agroal / pgagroal

High-performance connection pool for PostgreSQL
https://agroal.github.io/pgagroal/
BSD 3-Clause "New" or "Revised" License
681 stars 61 forks source link

Fix error message in `pgagroal-admin master-key` #428

Closed fluca1978 closed 6 months ago

fluca1978 commented 6 months ago

There is a wrong error message reporting pgexporter.

See #425

fluca1978 commented 6 months ago

My idea about the check for the master-key was a little different: do we care if the master key file is in place? We should care about the existance of any of the "users.conf" files, and warn the user about that.

jesperpedersen commented 6 months ago

I think that is more a documentation issue... if a user want to change master key they should know that all the user based files need to changed as well