Skip to content
This repository was archived by the owner on Dec 19, 2025. It is now read-only.

[ADMINAPI-1270] Add permissions in 6.X DB Dockerfile#386

Merged
jpardogrowthaccelerationpartners merged 1 commit intomainfrom
ADMINAPI-1270
Dec 1, 2025
Merged

[ADMINAPI-1270] Add permissions in 6.X DB Dockerfile#386
jpardogrowthaccelerationpartners merged 1 commit intomainfrom
ADMINAPI-1270

Conversation

@jagudelo-gap
Copy link

This pull request updates the Docker configuration for the Admin API service, focusing on improving security, environment flexibility, and feature enablement. The most important changes are grouped below by theme.

Security and Permissions:

  • Changed permissions for the PgSql scripts directory from 600 to 700 and set executable permissions for the migration script 3-run-adminapi-migrations.sh in Docker/Settings/V1/DB-Admin/pgsql/Dockerfile. This ensures scripts are executable by the owner, improving compatibility and security.

Configurable Build and Environment:

  • Added support for passing the ADMIN_API_VERSION as a build argument in the Docker Compose configuration, allowing easier version management during builds in Docker/V1/Compose/pgsql/compose-build-binaries.yml.

Feature Enablement and Environment Variables:

  • Enabled several features by default for the Admin API service, including Docker environment flags, Swagger documentation, and user registration, through new environment variables in the Compose file. This simplifies local development and testing.

@jagudelo-gap jagudelo-gap changed the title [ADMINAPI-1270] Added permissions in 6.X DB Dockerfile [ADMINAPI-1270] Add permissions in 6.X DB Dockerfile Dec 1, 2025
@jpardogrowthaccelerationpartners jpardogrowthaccelerationpartners merged commit 6dc5445 into main Dec 1, 2025
13 checks passed
@jpardogrowthaccelerationpartners jpardogrowthaccelerationpartners deleted the ADMINAPI-1270 branch December 1, 2025 20:24
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants