Skip to content

#6229 Fix improper output rendering#6230

Open
martgil wants to merge 25 commits into
masterfrom
issue-6229-fix-improper-output-rendering
Open

#6229 Fix improper output rendering#6230
martgil wants to merge 25 commits into
masterfrom
issue-6229-fix-improper-output-rendering

Conversation

@martgil
Copy link
Copy Markdown
Collaborator

@martgil martgil commented May 22, 2026

This PR fixes improper rendering of certain outputs rendered by the browser extension.

close #6229


Tests (delete all except exactly one):

  • Tests added or updated

To be filled by reviewers

I have reviewed that this PR... (tick whichever items you personally focused on during this review):

  • addresses the issue it closes (if any)
  • code is readable and understandable
  • is accompanied with tests, or tests are not needed
  • is free of vulnerabilities
  • is documented clearly and usefully, or doesn't need documentation

@martgil martgil requested a review from sosnovsky as a code owner May 22, 2026 08:14
@martgil martgil marked this pull request as draft May 22, 2026 08:14
@sosnovsky
Copy link
Copy Markdown
Collaborator

Hi @martgil, do you think this PR will be finished on this week so we can publish a new release next week with these fixes included?

@martgil
Copy link
Copy Markdown
Collaborator Author

martgil commented May 28, 2026

Hello @sosnovsky - Yes, it should be finished within this week. Not sure why CodeQL test fails. I'll review.

@martgil martgil marked this pull request as ready for review May 29, 2026 08:44
@martgil
Copy link
Copy Markdown
Collaborator Author

martgil commented May 29, 2026

Hi @sosnovsky - This one is ready for a review. Thank you!

Comment thread extension/manifest.json Outdated
@martgil martgil requested a review from sosnovsky June 1, 2026 05:29
@martgil
Copy link
Copy Markdown
Collaborator Author

martgil commented Jun 1, 2026

Hi @sosnovsky - I apologize for the delay. Hopefully its not too inconvenient to add this for the planned release.

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.

Review insecure rendered outputs

2 participants