Skip to content

Only print password length warning for 12 and 13 #1166

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

Merged
merged 1 commit into from
Dec 22, 2023

Conversation

tianon
Copy link
Member

@tianon tianon commented Dec 11, 2023

In 14+, the arbitrary length limitations have been removed from the PostgreSQL server (postgres/postgres@67a472d).

Updates #511
See #507 (comment)

In 14+, the arbitrary length limitations have been removed from the PostgreSQL server (postgres/postgres@67a472d).
@tianon
Copy link
Member Author

tianon commented Dec 11, 2023

I've got this as a draft because I don't love relying on PG_MAJOR in this script. 😞

Maybe it's OK because it's only for a warning that's harmless either way and only for the two oldest versions? On the other hand, PostgreSQL 13 is actively supported until Nov 2025. 😅

@tianon tianon marked this pull request as ready for review December 22, 2023 00:27
@tianon tianon merged commit ce5bf6e into docker-library:master Dec 22, 2023
@tianon tianon deleted the unlimited-passwords branch December 22, 2023 00:27
docker-library-bot added a commit to docker-library-bot/official-images that referenced this pull request Dec 22, 2023
Changes:

- docker-library/postgres@ce5bf6e: Merge pull request docker-library/postgres#1166 from infosiftr/unlimited-passwords
docker-library-bot added a commit to docker-library-bot/official-images that referenced this pull request Jan 4, 2024
Changes:

- docker-library/postgres@aa542a0: Merge pull request docker-library/postgres#1179 from infosiftr/less
- docker-library/postgres@d416768: Add `less` to Debian variants
- docker-library/postgres@ce5bf6e: Merge pull request docker-library/postgres#1166 from infosiftr/unlimited-passwords
martin-g pushed a commit to martin-g/docker-official-images that referenced this pull request Apr 3, 2024
Changes:

- docker-library/postgres@aa542a0: Merge pull request docker-library/postgres#1179 from infosiftr/less
- docker-library/postgres@d416768: Add `less` to Debian variants
- docker-library/postgres@ce5bf6e: Merge pull request docker-library/postgres#1166 from infosiftr/unlimited-passwords
# 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.

2 participants