Mathieu Pillard

Results 219 comments of Mathieu Pillard
trafficstars

`bleach` ensures tags are closed properly, so at least we don't have to worry about unbalanced tags. We allow: - `a` - `abbr` - `acronym` - `b` - `blockquote` -...

> But we need to do some refactoring to only do that in `sanitizeUserHTML` instead of the more global `sanitizeHTML`. The former doesn't have access to `DOMPurify` directly, hence the...

Before, on [dev](https://addons-dev.allizom.org/en-US/firefox/addon/ratopuse-weasp-vulturkal/versions/): ![Image](https://github.com/user-attachments/assets/8c530ff1-493c-48e6-aa8c-f20a6a0b50f0) After, once my patch lands, on dev: ![Image](https://github.com/user-attachments/assets/a7fd6d05-d355-4b9b-b8fe-74896ae3a85d)

Markdown makes it harder to reproduce, I fixed existing HTML content, as can be seen with https://addons-dev.allizom.org/en-US/firefox/addon/ratopuse-weasp-vulturkal/versions/

Is EFS mounted on `services` and `versioncheck` ? The function that is failing is checking permissions on various paths.

Is there an env variable or something other than the request URL I can use to detect we're on a `services` or `versioncheck` instance ?

We should have `ADDONS_SERVER_COMPONENT` nowadays - so we could add a check to prevent adding the `path` monitor in `front_heartbeat()` if `os.environ.get('ADDONS_SERVER_COMPONENT') in ('services-web', 'versioncheck-web')`.