Skip to content

Conversation

pjonsson
Copy link
Contributor

@pjonsson pjonsson commented Sep 3, 2025

The PostgreSQL documentation for packages
(https://www.postgresql.org/download/linux/ubuntu/) does not have a trailing slash.

@pjonsson pjonsson force-pushed the update-apt-repo-url branch from 94c6b57 to 3665f0d Compare September 3, 2025 16:39
@tianon
Copy link
Member

tianon commented Sep 3, 2025

I'm fine with dropping the trailing slash, but I do have a slight preference for not doing https specifically because man-in-the-middle here is typically a "transparent caching proxy" feature, not a bug, and is still protected by the PGP signature on the repository.

The PostgreSQL documentation for packages
(https://www.postgresql.org/download/linux/ubuntu/)
does not have a trailing slash.
@pjonsson pjonsson force-pushed the update-apt-repo-url branch from 3665f0d to abab72c Compare September 3, 2025 21:49
Copy link
Member

@tianon tianon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, thank you!

@tianon tianon merged commit bbf07d5 into docker-library:master Sep 3, 2025
38 checks passed
docker-library-bot added a commit to docker-library-bot/official-images that referenced this pull request Sep 4, 2025
Changes:

- docker-library/postgres@bbf07d5: Merge pull request docker-library/postgres#1359 from pjonsson/update-apt-repo-url
- docker-library/postgres@abab72c: debian: align postgres repo url with docs
@pjonsson pjonsson deleted the update-apt-repo-url branch September 4, 2025 08:12
docker-library-bot added a commit to docker-library-bot/official-images that referenced this pull request Sep 5, 2025
Changes:

- docker-library/postgres@5ab2c9d: Update 18 to 18rc1, trixie 18~rc1-1.pgdg13+1, bookworm 18~rc1-1.pgdg12+1
- docker-library/postgres@bbf07d5: Merge pull request docker-library/postgres#1359 from pjonsson/update-apt-repo-url
- docker-library/postgres@abab72c: debian: align postgres repo url with docs
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.

2 participants