Skip to content

refactor: fix outdated dependencies and deprecated apt-key usage in d… #6769

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

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

Conversation

Aminbaklouti
Copy link

@Aminbaklouti Aminbaklouti commented Apr 25, 2025

This is a ...

  • New feature
  • Improvement
  • Bug fix
  • Others

Motivation

Description

This PR addresses critical issues in the Dockerfile to improve security, compatibility, and maintainability:

Removed invalid PostgreSQL client version 17 (unsupported, replaced with valid v16).

Replaced deprecated apt-key with modern keyring handling for PostgreSQL repository authentication.

Set BEFORE_PACK_NOCOBASE to a no-op default to avoid unnecessary file listing.

Impact:

Ensures compatibility with PostgreSQL 16.

Aligns with Debian best practices for secure package management.

Streamlines the build process.

Testing:

Verified Docker build completes successfully.

Confirmed PostgreSQL client installation works as expected.

This PR resolves immediate technical debt without disrupting existing workflows.

Related issues

Showcase

Changelog

Language Changelog
🇺🇸 English
🇨🇳 Chinese

Docs

Language Link
🇺🇸 English
🇨🇳 Chinese

Checklists

  • All changes have been self-tested and work as expected
  • Test cases are updated/provided or not needed
  • Doc is updated/provided or not needed
  • Component demo is updated/provided or not needed
  • Changelog is provided or not needed
  • Request a code review if it is necessary

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

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