Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Remove rate limiting on NickServ/ChanServ #240

Closed
kegsay opened this issue Oct 17, 2016 · 3 comments
Closed

Remove rate limiting on NickServ/ChanServ #240

kegsay opened this issue Oct 17, 2016 · 3 comments
Assignees
@lukebarnard1
Labels
api-change matrix.org-support Matrix.org specific problem possibly unrelated to the bridge p1

Comments

@kegsay
Copy link
Member

kegsay commented Oct 17, 2016 •
edited
Loading

Requires:

  • The AS reg file has a flag to say turn off rate limiting (default: rate limiting is on).
  • Synapse to honour the reg file
  • The bridge to flip rate limiting off on reg file generation.
@kegsay kegsay mentioned this issue Oct 17, 2016
Closed
@ara4n ara4n added the p1 label Oct 17, 2016
@lukebarnard1
Copy link
Contributor

lukebarnard1 commented Oct 18, 2016 •
edited
Loading

lukebarnard1 pushed a commit to matrix-org/synapse that referenced this issue Oct 18, 2016
Allow Configurable Rate Limiting Per AS
5b54d51 
This adds a flag loaded from the registration file of an AS that will determine whether or not its users are rate limited (by ratelimit in _base.py). Needed for IRC bridge reasons - see matrix-org/matrix-appservice-irc#240.
@lukebarnard1 lukebarnard1 mentioned this issue Oct 18, 2016
Merged
@lukebarnard1 lukebarnard1 added the matrix.org-support Matrix.org specific problem possibly unrelated to the bridge label Oct 26, 2016
@lukebarnard1
Copy link
Contributor

lukebarnard1 commented Oct 26, 2016 •
edited
Loading

Added support label as the freenode bridge at matrix.org will be redeployed tomorrow on 0.6.0 to test that this works. If it does, I'll close this issue.

Blocked on doing another redeploy of matrix.org synapse ( #257 )

@kegsay
Copy link
Member Author

kegsay commented Nov 1, 2016

It does work.

@kegsay kegsay closed this as completed Nov 1, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lukebarnard1 lukebarnard1

Labels
api-change matrix.org-support Matrix.org specific problem possibly unrelated to the bridge p1
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@ara4n @lukebarnard1 @kegsay