Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hi,
From what I understand, if
stats_users
specified in configuration, it needsto be present in
auth_file
to be able to connect to the admin db. This is thecase even if
stats_users
could be authenticated viaauth_user
+auth_query
for all other "regular" databases. Looks like it happens becausethe admin db is a special "fake" database without an
auth_user
. If pgbouncerwould be able to authenticate
stats_users
connecting to the admin db usingauth_query
as well, this will be helpful because one doesn't need to maintainanother set of credentials for a monitoring user.
From the first glance there are just few bits missing to implement this. E.g. a
default_auth_db
to connect to for authentication purposes in case of admindb. Does this implementation make sense?
This idea also partially intersects with #569, but instead of restricting this
patch for admin databases it handles them differently.