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 a security issue w/ file names and git publishing... #322

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

Conversation

jmgurney
Copy link

Don't use shell, and ensure that file names, like:
"; touch /tmp/securityissue; "

Won't cause an issue...

Don't use shell, and ensure that file names, like:
"; touch /tmp/securityissue; "

Won't cause an issue...
@jmgurney
Copy link
Author

jmgurney commented Dec 14, 2016

Some of these failures have nothing to do w/ my code. I cannot control/install asciidocapi on the machine to fix job 99.1. Some of the failures in job 99.5 are not mine. Job 99.6 failure is related to failure to maintain and again, unrelated to my changes.

@Descent098
Copy link
Member

I can't verify if the checks are failing due to the issues resolved in #329 because travis CI is down. I will pull and rebase this branch off the current 0.9.0 branch and if it passes the tests off that branch when travis is back up I will merge it in.

@Descent098 Descent098 self-assigned this Mar 21, 2020
@Descent098 Descent098 added the Maintainer Task Any task that needs to be completed by a maintainer label Mar 21, 2020
@Descent098 Descent098 added this to TODO in V0.9.0 Mar 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Maintainer Task Any task that needs to be completed by a maintainer
Projects
V0.9.0
  
TODO
Development

Successfully merging this pull request may close these issues.

None yet

2 participants