Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: Avoid TypeError for null values on CUSTOM_ELEMENT_HANDLING config. #898

Merged
merged 1 commit into from
Jan 5, 2024

Conversation

cpmotion
Copy link
Contributor

@cpmotion cpmotion commented Jan 5, 2024

ISSUE-897

Summary

See issue. Change made in utils and test added.

Tasks

Not sure if there's a better way to inverse assert in the test. Please advise if so. Tested locally to pass and fail. It "works" but kind of a workaround for QUnit it seems...

@cpmotion
Copy link
Contributor Author

cpmotion commented Jan 5, 2024

Just realized saving in test-suite applied prettier fixes. I guess prettier is not running in pipeline to validate against config. Hmm, I'm going to undo those lines for this specific commit and update pr... Updated.

@cure53
Copy link
Owner

cure53 commented Jan 5, 2024

This is great, thanks you!

@cure53 cure53 merged commit 486673f into cure53:main Jan 5, 2024
8 checks passed
@cpmotion
Copy link
Contributor Author

cpmotion commented Jan 5, 2024

@cure53 any chance we could push a patch(3.0.8) out asap? We have a specific scenario where we're picking up latest v3 and that would really help a currently deployed product version. Thank you!

@cure53
Copy link
Owner

cure53 commented Jan 5, 2024

Yep, we needed to wait for feedback on the other issue, it just came in and we are ready to release.

@cure53
Copy link
Owner

cure53 commented Jan 5, 2024

This release should fix it: https://github.com/cure53/DOMPurify/releases/tag/3.0.8

@cpmotion
Copy link
Contributor Author

cpmotion commented Jan 5, 2024

Awesome thanks!

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.

2 participants