[FEATURE REQ] azure-identity-extensions for PostgreSQL is inneficient

[paulaustin-automutatio]

https://github.com/Azure/azure-sdk-for-java/tree/main/sdk/identity/azure-identity-extensions

I just wanted to let you know that the way the above is implemented for PostgreSQL is inefficient.

This is due to the way that the postgreSQL driver creates an instance of the authenticator plugin for each connection request.

This means that a new token is requested for each connection request.

I'm just reporting this as a courtesy as I'm not going to be using this code

[github-actions]

@billwert @g2vinay

[github-actions]

Thank you for your feedback. Tagging and routing to the team member best able to assist.

[g2vinay]

@paulaustin-automutatio
Thank you for reporting this, would be great if you could point out to the code as well.

Assigning to @saragluna to take a look.

[paulaustin-automutatio]

Have a look at com.azure.identity.extensions.jdbc.postgresql.AzurePostgresqlAuthenticationPlugin

Put a breakpoint in the constructor and you will see that it is created for every new database connection. That means it has to get a new token every time as that instance can't be cached. PostgreSQL driver provides no way to cache it.

You'd have to do some kind of caching using a static variable.