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

Address security vulnerability in rubyzip dependency #602

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

waterjump
Copy link

The rubyzip gem version 1.2.1 contains a security vulnerability allowing
absolute path traversal. More details can be found here:

rubyzip/rubyzip#369

This change addresses the issue by specifying a rubyzip version greater
than or equal to 1.2.2.

Solves issue #599

The rubyzip gem version 1.2.1 contains a security vulnerability allowing
absolute path traversal.  More details can be found here:

rubyzip/rubyzip#369

This change addresses the issue by specifying a rubyzip version greater
than or equal to 1.2.2.

Solves issue randym#599
@why-el
Copy link

why-el commented Sep 11, 2018

@waterjump any chance you release a new version with this change? It's a pretty serious one.

@waterjump
Copy link
Author

@why-el Seems like bumping it to 3.0.1 would be a good idea. I'd like to confirm with the gem owner because people tend to do this differently from time to time and there's nothing in the README about contribution guidelines etc.

@why-el
Copy link

why-el commented Sep 11, 2018

Ok, thanks the prompt response. Up to @randym then.

@noniq
Copy link
Collaborator

noniq commented Sep 11, 2018

See also #536

@sullyvannunes
Copy link

I am facing this same problem with rubyzip version.
is there any update about this issue?

@courtsimas
Copy link

Ping. What's the latest?

@waterjump
Copy link
Author

@courtsimas We are waiting on feedback from @randym regarding version bump.

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.

5 participants