Admin API for server notice: consistently bypass rate limits

[MatMaul]

Signed-off-by: Mathieu Velten matmaul@gmail.com

A lot of the underlying calls in the server notice code are already using ratelimit=False, but some are missing leading to still being rate limited when using the API in bulk.

Pull Request Checklist

• Pull request is based on the develop branch
• Pull request includes a changelog file.
• Pull request includes a sign off
• Code style is correct
  (run the linters)

[DMRobertson]

Thanks Mathieu. How confident are we that this covers everything, i.e. that we haven't missed any other handler calls?

[MatMaul]

Thanks Mathieu. How confident are we that this covers everything, i.e. that we haven't missed any other handler calls?

I am now able to use this API in bulk for all the users of a test homeserver with 1000 test users, while I couldn't before, so I am pretty confident it is fine now.
I also had a look at the whole code in this handler file since it's pretty small and couldn't spot anything else.

[DMRobertson]

Force-merging because CI passed on the previous commit, and the only change since then is to the changelog.