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.

Sign up for GitHub

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

Jump to bottom

fix: sanitize jsURL in toolbar #25993

Closed

mariusandra wants to merge 1 commit into master from sanitize-js-url

Collaborator

mariusandra commented Nov 5, 2024

Problem

We've gotten a report this variable can be misused.

Changes

Make sure it's a URL

Does this work well for both Cloud and self-hosted?

Yes

How did you test this code?

WIP


          fix: sanitize jsURL in toolbar

2f3d1b1

Contributor

github-actions bot commented Nov 5, 2024

Size Change: +87 B (+0.01%)

Total Size: 1.15 MB

ℹ️ View Unchanged

Filename	Size	Change
`frontend/dist/toolbar.js`	1.15 MB	+87 B (+0.01%)

_{compressed-size-action}

Collaborator

timgl commented Nov 5, 2024

Superseded by #25995

timgl closed this

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

Labels

None yet