Just updated, getting 429 errors in Edge

AI-generated summary

A user reported receiving 429 errors in Edge when accessing topics on blenderartists.org. The error occurs due to the server only accepting one ‘JS error report’ per minute per IP david. To fix the issue, a pull request was created to disable the copy codeblocks feature for older Edge versions, which was merged and backported into beta by joffreyjaffeux. This fix resolves the 429 error and prevents the browser from crashing, but it means the code copy button will not work for affected users. joffreyjaffeux notes that recent Edge versions are not affected, and this solution is acceptable given the circumstances.

A user just reported not being able to access any topics on blenderartists.org in Edge. He sees the following message in the console:

HTTP429: TOO MANY REQUESTS - The server is refusing to service the request because too many requests have been submitted by the client.
(XHR)POST - https://blenderartists.org/logs/report_js_error

I just tested here on Meta too and got the same error, as well as:

SCRIPT5022: SyntaxError
application-b3556b113cb3143e8f644cc27166bfe827925980ba6e06d67060c311601cc4c3.br.js (1,2404988)

which leads to:

           if (!siteSettings.show_copy_button_on_codeblocks || isIE11) {
            return;
          }
3 Likes

Looking at this thanks

4 Likes

So this is only “previous edge”, recent edge is working fine.

Probably something is not accessible in this edge, will see if we can polyfill or just disable support of copy codeblocks for “previous edge” in this version.

2 Likes

That should fix it:

https://github.com/discourse/discourse/pull/9529

Asking to a coworker to double check it fixes the issue, and I will then merge it and backport it into beta. It means this feature won’t be working for this user. We could probably make it work but given recent edge works I think this is OK, this fix ensures browser won’t crash if they don’t support any of the css selectors used in this feature.

4 Likes

I think he can live without the code copy button, but this will fix the 429 issue too?

The server only accepts one ‘JS error report’ per minute per IP, so that’s why the 429 appeared. Nothing to worry about, it won’t affect anything. Unfortunately we can’t suppress HTTP errors from the console.

4 Likes

Merged and backported into beta

Thanks for report :+1:

3 Likes