Skip to content

fix(apps): prevent reflected XSS via un-encoded response#3737

Merged
alan-agius4 merged 1 commit into
angular:mainfrom
josephperrott:fix-dns-redirecting-xss
Jun 5, 2026
Merged

fix(apps): prevent reflected XSS via un-encoded response#3737
alan-agius4 merged 1 commit into
angular:mainfrom
josephperrott:fix-dns-redirecting-xss

Conversation

@josephperrott
Copy link
Copy Markdown
Member

Fixes a reflected XSS vulnerability in the dns-redirecting cloud function by properly setting the Content-Type to text/plain before returning a 404 response.

Validated via testing and manual PoC verification.

@josephperrott josephperrott requested a review from alan-agius4 June 4, 2026 22:05
@josephperrott josephperrott added the action: merge The PR is ready for merge by the caretaker label Jun 4, 2026
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the dnsRedirecting function in apps/functions/dns-redirecting/index.ts to explicitly set the response content type to 'text/plain' when returning a 404 error message. There are no review comments, and I have no further feedback to provide.

@alan-agius4 alan-agius4 merged commit 65d9d54 into angular:main Jun 5, 2026
16 checks passed
@alan-agius4
Copy link
Copy Markdown
Contributor

This PR was merged into the repository. The changes were merged into the following branches:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

action: merge The PR is ready for merge by the caretaker

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants