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

Cleans up deprecated apt-key usage #408

Merged
merged 4 commits into from
May 16, 2022
Merged

Cleans up deprecated apt-key usage #408

merged 4 commits into from
May 16, 2022

Conversation

tbollinger
Copy link
Contributor

Cleans up deprecated apt-key calling and piping usage through using gpg directly.

@driesvints
Copy link
Member

There's no explanation here why this is needed.

@tbollinger
Copy link
Contributor Author

Apt-key won't be around next Ubuntu release, but it "works" for now so I suppose the problem can be dealt with during the creation of those containers in a few years.

The main reason for this PR is that the advice given by these deprecation warnings to add keys to /etc/apt/trusted.gpg.d/ is not considered best practice. While I don't see cleaning up these warnings as a necessity, I think it is useful simply in preventing people from performing a half-measure and doing the wrong thing based on the advice given by these warnings.

Related content:
https://blog.cloudflare.com/dont-use-apt-key/
https://wiki.debian.org/DebianRepository/UseThirdParty#OpenPGP_Key_distribution

@taylorotwell taylorotwell merged commit 4b90a06 into laravel:1.x May 16, 2022
@RuneSphinx RuneSphinx mentioned this pull request Sep 15, 2022
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.

None yet

3 participants