429 too many emoji requests 😥

Support

AI-generated summary

429 too many emoji requests :sad_but_relieved_face:

A discussion on a recurring issue where users receive a 429 Too Many Requests error when loading emojis in the post editor. Fredo initially reports the issue, suggesting it may be due to Discourse considering multiple requests to load emojis as a potential DDOS attack.

SAM points out that combining emojis into a sprite would be unnecessary with HTTP/2 and asks for clarification on whether the issue occurs on meta.discourse.org or another hosted site. Codinghorror also requests clarification, which Fredo provides, stating the issue occurs on a hosted site (https://forums.sketchup.com/) and can be reproduced by quickly scrolling through emojis.

SAM assigns the issue to himself and proposes two solutions: whitelisting emoji requests from the CDN and centralizing emoji storage across sites. Codinghorror also suggests centralizing avatar paths. SAM later updates that while some improvements have been made, centralizing emoji storage remains a significant task that has not been scheduled.

Canapin ultimately considers the issue fixed due to a lack of recent occurrences.

1

After opening the emoji selector in the post editor, discourse throws users out once they load a page again (like after submitting a post).

429 Too Many Requests

Your IP address has made too many requests to this service over a short amount of time. Please wait a few minutes and try again.

If you are behind a proxy or coming from a large company, you and many other users may appear to be the same person. Please let us know of recurring problems by email: team@discourse.org

It seems it was not the user trying to DDOS discourse, but discourse itself. If there is a way, maybe distinct requests to resources (like emojis) should not be considered? Or all emojis should be combined into a sprite?

2

Very little point in that cause we have HTTP/2

Can you clarify, is this happening on meta.discourse.org or on some other site we are hosting?

If it’s on meta then yes we got to add a bypass to our special rate limiting here for emojis, discourse_docker/templates/web.ratelimited.template.yml at master · discourse/discourse_docker · GitHub is too indescriminent.

One plan we have is to shift to a global rate limit in the APP so this would be totally sidestepped and then the NGINX rate limit can be way higher.

3

You will need to clarify this part first @Fredo.

4

It’s hosted by discourse (https://forums.sketchup.com/). But you have to scroll quickly through all the emojis to reproduce it.

5

FTR I was able to reproduce it on https://forums.sketchup.com by scrolling through the emojis.

7

I am leaving this assigned to me for now, we do plan to improve this in 2 ways

  1. whitelist emoji requests from the CDN so they do not participate in the rate limiting

  2. centralize emoji storage across all our sites so there is only one CDN for emojis.

8

We need to centralize emoji paths and avatar paths (on Discourse hosting). So this is a to-do we still need to get to.

9

Temos feito muito aqui para melhorar a situação: regras mais claras em nossa CDN e limites mais flexíveis para emojis.

A mudança fundamental de fazer com que toda a nossa hospedagem use um único local para todos os emojis ainda não foi implementada, e não consigo prever que isso seja agendado em um futuro próximo, devido à magnitude de uma mudança dessa natureza.

11

Como não houve ocorrência recente de tal problema, acho que podemos considerá-lo resolvido. :slight_smile: