Skip to content

#397 - Add support for Azure Postgresql Server with Managed Identities#399

Open
alexmaie wants to merge 1 commit into
hangfire-postgres:masterfrom
alexmaie:master
Open

#397 - Add support for Azure Postgresql Server with Managed Identities#399
alexmaie wants to merge 1 commit into
hangfire-postgres:masterfrom
alexmaie:master

Conversation

@alexmaie

Copy link
Copy Markdown

In Azure we have the concept of managed identities which allows identities to connect without a password. This is handled by requesting a token from azure for a specific resource. An identity can be an azure resource ( VM, app service, etc ) or a user.

Work:

  • created a new assembly in order to isolate the nuget requirements for Azure Tokens: Hangfire.PostgreSql.Azure
  • implemented a factory so that users can easily plugin and override settings
  • introduced Hangfire.PostgreSql.Azure.Tests for the new assembly. Added a test to make sure that we can connect to a database by using the setup
  • updated readme

* created a new assembly in order to isolate the requirements for Azure Tokens: Hangfire.PostgreSql.Azure
* implemented a factory so that users can easily plugin
* introduced Hangfire.PostgreSql.Azure.Tests for the new assembly tests
* updated readme
@alexmaie

Copy link
Copy Markdown
Author

Hi friends,

Any feedback on this PR :D ?

@azygis

azygis commented Apr 26, 2025

Copy link
Copy Markdown
Collaborator

Hey, this will have to wait, but feel free to just use what you have. I'm currently in the process of major refactoring and will release v2. This package will probably go out together with it.

In terms of feedback, brief look doesn't raise any pain points. Good work. Will need to check the branch out when it's time and see how it behaves. Not sure how the versioning should work when it's kinda same nuget push. Will have to think about it.

@alexmaie

Copy link
Copy Markdown
Author

Hi @azygis

thanks for the reply. I understand that this topic does not have a high priority right now. I just wanted to get some feedback sooner then later why I have everything fresh in my head in case some changes are needed.

If it's the same nuget push for versioning, I would suggest to keep the versions in sync between them, similar to Hangfire.Core and Hangfire.SqlServer

@alexmaie

Copy link
Copy Markdown
Author

Hi friends,
Wanted to circle back to this PR since it was created for some time now, and would like to merge it before I forget everything related to it :D

@vicentegnz

Copy link
Copy Markdown
Contributor

Hi @azygis @alexmaie , are there any plans to integrate this? It would be useful to have it, as at the moment we have to use a “custom hangfire” to connect it to Azure Managed Identity...

If I can help speed things up, review, or whatever else is needed...

@alexmaie

Copy link
Copy Markdown
Author

Hi, I can fix the conflicts at some point this week, but it would be good to know @azygis if there is time to get this PR merged in the near future so that I don't have to fix the conflicts multiple times

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants