Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Use Retry-After header in addition to retry_after_ms #7138

Closed
sents opened this issue Mar 25, 2020 · 2 comments
Closed

Use Retry-After header in addition to retry_after_ms #7138

sents opened this issue Mar 25, 2020 · 2 comments

Comments

@sents
Copy link

sents commented Mar 25, 2020

Description:

When exceeding the rate limit synapse sends a 429, M_LIMIT_EXCEEDED response with the json-attribute retry_after_ms. There seems to be a standard response header for this: Retry-After
Using this header in addition to the retry_after_ms attribute would make it easier to handle the retries in common http libraries.

@anoadragon453
Copy link
Member

Nice idea! Though this is more of an issue for the Matrix Specification repo than Synapse. Once it's cleared in the spec it can be added to Synapse :)

@sents
Copy link
Author

sents commented Mar 27, 2020

Ok, then I will open this issue in the spec repo. Thank you!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants