Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add support for access token authentication to an Azure Postgres Flexible Server - Postgres scaler #5823

Closed
Ferdinanddb opened this issue May 23, 2024 · 3 comments · Fixed by #5589
Assignees
Labels
feature-request All issues for new features that have not been committed to needs-discussion stale All issues that are marked as stale due to inactivity

Comments

@Ferdinanddb
Copy link
Contributor

Proposal

The Postgres scaler only supports basic authentication through username and password, but it is now possible to authenticate to a Postgres server hosted in the cloud by using "access token".

The purpose of this issue is to add support to enable access token authentication within the Postgres scaler by extending it, possibly with additional parameters.

Use-Case

In Azure, a managed identity does not have a password associated to it, but can obtain short-lived credentials (called access tokens) to connect to certain resources. It is possible for an Azure managed identity to connect to an Azure Postgres Flexible server.

Is this a feature you are interested in implementing yourself?

Yes

Anything else?

I created this PR: #5589. I tested it in my own Azure subscription to see if it works with my own use case (related to Apache Airflow) and it worked.

Copy link

stale bot commented Jul 26, 2024

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 7 days if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale All issues that are marked as stale due to inactivity label Jul 26, 2024
@Ferdinanddb
Copy link
Contributor Author

The doc of this new feature has been updated via this PR in keda-docs.

Thank you very much for your help on this @JorTurFer, much appreciated!

@JorTurFer
Copy link
Member

You're welcome :)
Thanks for this nice improvement! We will cut a release during the week, so you can expect it soon :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature-request All issues for new features that have not been committed to needs-discussion stale All issues that are marked as stale due to inactivity
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants