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

Update README.md #9

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

Conversation

JorisVanEijden
Copy link

@JorisVanEijden JorisVanEijden commented Jan 20, 2021

Document security downside of using this package.
I'm not aware of an easy solution for updating a vulnerable package with this package in place but if someone does it would be good to document that here too.

Edit: Something like

composer require drupal/core:^8.0
composer remove drupal/core-recommended
composer update pear/archive_tar

would work but you'd have to switch back if you want to keep the benefits of core-recommended.

Document security downside of using this package.
I'm not aware of an easy solution for updating a vulnerable package with this package in place but if someone does it would be good to document that here too.
@amansrivastava
Copy link

This will solve the issue for the time being.
composer req pear/archive_tar:"1.4.12 as 1.4.11"

@SOMERSOFTAndrewCrook
Copy link

SOMERSOFTAndrewCrook commented Jun 10, 2022

In a similar vein
"guzzlehttp/guzzle": "6.5.7 as 6.5.6",
Failure to strip the Cookie header on change in host or HTTP downgrade

@RobinHoutevelts
Copy link

RobinHoutevelts commented Jun 13, 2022

https://www.drupal.org/node/3285240

Seems like Drupal 9.5 will allow patch updates on dependencies 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants