500 Errors on Rate Limit When Using the Admin API Via Keys

[sam (Sam Saffron)]

OK this is fixed per:

We should never leak out exceptions in our controller tests

I just finished pushing a rather gnarly change per: https://github.com/discourse/discourse/commit/49ed382c2a7ddd11756e552e094319861418cd7f It basically replaces this pattern we used previously in tests: expect do get :index, format: :json end.to raise_error(Discourse::NotLoggedIn) With this get :index, format: :json expect(response.status).to eq(403) I made this change so our test environment is much more inline with our production environment. Previously we were leaking out a few exc…

But it will take a bit for it to land cause I need to clean plugin tests.

The particular reason you are getting rate limited is:

discourse/discourse/blob/main/config/discourse_defaults.conf#L177

if you are self hosting you can raise DISCOURSE_MAX_ADMIN_API_REQS_PER_KEY_PER_MINUTE to a higher number. We introduced this limit to protect from unintentional API abuse.