Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Conversation

@pull
Copy link

@pull pull bot commented Dec 17, 2025

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 馃挅 Please sponsor : )

Before this commit:
`lxml.html.clean.Cleaner` striped all ARIA attributes (e.g., aria-label, aria-hidden, role) by default.
This is because they were not included in the "safe attributes" whitelist.

This behavior caused sanitized HTML to lose semantic meaning for assistive technologies
(screen readers), making the output non-compliant with WCAG standards.

After this commit:
ARIA attributes are added to `defs.safe_attrs` so the Cleaner will no longer filter them out.
This change ensures that developers using lxml for sanitization will not accidentally strip
accessibility features while maintaining protection against XSS.

Closes https://bugs.launchpad.net/bugs/2136144
@pull pull bot locked and limited conversation to collaborators Dec 17, 2025
@pull pull bot added the 猡碉笍 pull label Dec 17, 2025
@pull pull bot merged commit 4153564 into sysfce2:master Dec 17, 2025
5 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

Morty Proxy This is a proxified and sanitized view of the page, visit original site.