apache / pulsar-manager

Apache Pulsar Manager
https://pulsar.apache.org/
Apache License 2.0
516 stars 245 forks source link

fix init_db.sql to also take variables from environment #556

Closed Mortom123 closed 6 months ago

Mortom123 commented 6 months ago

Motivation

The init script inside the docker container did not depend on the environment variables that were set in the backend (USERNAME / PASSWORD). This can lead to problems when trying to connect to the database and expecting the same credentials as set in the environment.

No errors where visible because postgre "trusts" connections from localhost by default, hence the backend did not have to authenticate / was allowed to authenticate incorrectly.

Modifications

The first SQL command to alter the role is made dependent on the environment variables.

Verifying this change