-
Notifications
You must be signed in to change notification settings - Fork 685
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
Update configAggregator.js Sanetize DOM Text Interpreted As HTML #4360
base: develop
Are you sure you want to change the base?
Conversation
Hii @glo82145 Could Team Please Review This PR |
If your PR is missing information, check against the original template here. At a minimum you must have the section headers from the template and provide some information in each section. |
Hii @glo82145 Could Team Please Review This PR |
Hii @engcom-Hotel @engcom-Charlie Could Team Please Review This PR also if possible |
Hey @glo42707 we request you to please look into this. |
Hii @glo82145 Could you please Review this PR |
Hii @glo82145 |
Hey @glo42707 request you to please look into this As I have updated all necessary Changes of there is still it fail Could you Provide some suggestions on it |
Ping @glo82145 |
Hii @engcom-Hotel @engcom-Charlie Could Team Please ping Reviewers again |
@Shivam7-1 team was reviewing your code and it seems some unit test is failing for same, however let me remind my team for updating which you need to check. Thanks |
Hii @glo42707 Thanks For Response i had updated Dependencies with Required File If there is issue let me know and why changes are Expected to Resolve of there anything Thanks |
Please check these failing test cases @Shivam7-1 try merging latest develop branch with your branch |
Hii @glo42707 Could you please Remind Team again if possible |
PR test fail is still there, however we are looking into cause of these error and will update soon |
@glo82145 okay let me know if anything required |
@Shivam7-1 hence request to please allow some time and based on priorities of ticket, it will also get reviewed and merged respectively. |
Hii @glo42707 Thanks For Response |
Description
Sanitize HTML content using DOMPurify before returning it in richContent. This ensures safe rendering of potentially untrusted HTML, protecting against XSS attacks. The innerHTML of the first child node is sanitized before being returned, improving security.