Skip to content

Security patch against inserted scripts #73

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

Closed
wants to merge 2 commits into from
Closed

Security patch against inserted scripts #73

wants to merge 2 commits into from

Conversation

hugotiburtino
Copy link
Contributor

Besides the security patch, there is a small suggestion to the README, just in case the dependencies are updated later.

With the current conditions, it is possible to insert a malicious
script into the pre and the head. The additions suggested at this
commit may not be elegant but at least they avoid such kind of
attack. Refactor maybe needed afterwards.

Resolves: #72
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.

1 participant