Skip to content

Validate rel attribute conflict with link_rel#117

Merged
messense merged 1 commit intomessense:mainfrom
gghez:fix/rel-attr-panic
Mar 24, 2026
Merged

Validate rel attribute conflict with link_rel#117
messense merged 1 commit intomessense:mainfrom
gghez:fix/rel-attr-panic

Conversation

@gghez
Copy link
Copy Markdown
Contributor

@gghez gghez commented Mar 24, 2026

Passing rel in attributes while link_rel is set (the default) causes
ammonia to panic. This validates the conflict upfront and raises a ValueError
with a message pointing to link_rel=None.

Fixes #36

@messense messense merged commit 9fdfa76 into messense:main Mar 24, 2026
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

PanicException if allowed attribute is missing

2 participants