Skip to content

Commit 3dae0b8

Browse files
authored
[DPE-7594] Add custom users to pg_hba filter (#1073)
* Add custom users to pg_hba filter * Fix unit test
1 parent 9d270e6 commit 3dae0b8

File tree

2 files changed

+11
-4
lines changed

2 files changed

+11
-4
lines changed

src/charm.py

Lines changed: 10 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2174,9 +2174,16 @@ def relations_user_databases_map(self) -> dict:
21742174
})
21752175
return user_database_map
21762176
try:
2177-
for user in sorted(
2178-
self.postgresql.list_users_from_relation(current_host=self.is_connectivity_enabled)
2179-
):
2177+
for user in self.postgresql.list_users(current_host=self.is_connectivity_enabled):
2178+
if user in (
2179+
"backup",
2180+
"monitoring",
2181+
"operator",
2182+
"postgres",
2183+
"replication",
2184+
"rewind",
2185+
):
2186+
continue
21802187
user_database_map[user] = ",".join(
21812188
sorted(
21822189
self.postgresql.list_accessible_databases_for_user(

tests/unit/test_charm.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2958,7 +2958,7 @@ def test_relations_user_databases_map(harness):
29582958
assert harness.charm.relations_user_databases_map == {}
29592959

29602960
# Test when there are relation users in the database.
2961-
_postgresql.list_users_from_relation.return_value = {"user1", "user2"}
2961+
_postgresql.list_users.return_value = ["user1", "user2"]
29622962
_postgresql.list_accessible_databases_for_user.side_effect = [{"db1", "db2"}, {"db3"}]
29632963
assert harness.charm.relations_user_databases_map == {"user1": "db1,db2", "user2": "db3"}
29642964

0 commit comments

Comments
 (0)