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: sanitize data for xss #450

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

guncebektas
Copy link
Member

this is a possible fix for #331
I'm using this on production without a problem.

@guncebektas
Copy link
Member Author

hi @jankapunkt can you please review the PR? It's a security patch so any suggestion will be great.

@jankapunkt
Copy link
Member

Hey @guncebektas this is not forgotten, we just discovered during #448 that there is a branch diversion between master and devel (master is 8 commits ahead and 3 commits behind at the same time). Once we figure this out we can merge this into whatever the most recent branch will be (I think devel).

This will also go into an RC release so everybody can safely test on their setups.

client/main.js Outdated Show resolved Hide resolved
@StorytellerCZ StorytellerCZ linked an issue Nov 27, 2023 that may be closed by this pull request
@jankapunkt jankapunkt changed the base branch from master to devel November 28, 2023 07:19
@ricaragao
Copy link
Member

@jankapunkt , did you have time to fix the conflicts? I'd like to prepare this package to support Meteor 3, but I prefer wait for this merge before. What do you think?

@jankapunkt jankapunkt changed the base branch from devel to master June 14, 2024 11:38
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.

Cross-Site Scripting (XSS)
3 participants