vttranlina opened a new pull request, #2292: URL: https://github.com/apache/james-project/pull/2292
When I tried to refactor PostgresExtension (Using PoolBackedPostgresConnectionFactory for all factories. Make it similar to prod code ([PostgresCommonModule](https://github.com/apache/james-project/blob/cfeba0162d4fa7c8549069649976f2cac45b628b/server/container/guice/postgres-common/src/main/java/org/apache/james/modules/data/PostgresCommonModule.java) ) ) I realized that `PoolBackedPostgresConnectionFactory` doesn't set attribute `app.current_domain` when Domain is empty. => Failed on some test cases with enableRLS + emptyDomain username ### How to fix it? set domain attribute value even when empty domain, make it similar to that we did in [DomainImplPostgresConnectionFactory](https://github.com/apache/james-project/blob/701790ec040011269640e13e416c5283ad649322/backends-common/postgres/src/main/java/org/apache/james/backends/postgres/utils/DomainImplPostgresConnectionFactory.java#L37) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@james.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: notifications-unsubscr...@james.apache.org For additional commands, e-mail: notifications-h...@james.apache.org
