Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix CVE command injection #15

Merged

Conversation

tomtaylor
Copy link
Contributor

@tomtaylor tomtaylor commented Nov 22, 2022

@tomtaylor
Copy link
Contributor Author

I currently am the only owner of this Rubygem, but am unable to write to this repo since leaving Newspaper Club many years ago. I'd like to get this fix merged and released and transfer ownership to someone who can manage this project. Is that you @Michael-Kurt or @jaisoncoelho? If not, I'll fork the repo.

@Michael-Kurt
Copy link

Hi @tomtaylor! You can assign that PR/fix to @jaisoncoelho for review and merge. Thanks for reaching out.

@tomtaylor
Copy link
Contributor Author

Thanks @Michael-Kurt - unfortunately I'm not able to assign anything in this repo.

@Michael-Kurt
Copy link

No problem. I will assign it. Thanks @tomtaylor

@jaisoncoelho jaisoncoelho merged commit 54d4e01 into newspaperclub:master Mar 3, 2023
@jaisoncoelho
Copy link
Contributor

jaisoncoelho commented Mar 3, 2023

@tomtaylor I'd like to thank you for let us aware regarding the CVE.
We'd like to get responsible for the gem available on RubyGems (https://rubygems.org/gems/pdf_info). Let us know if it sounds good to you, and then we can proceed with the repository transfer.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants