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

Support Active Directory Authentication for MS SQL in native-image #36587

Open
turing85 opened this issue Oct 19, 2023 · 9 comments
Open

Support Active Directory Authentication for MS SQL in native-image #36587

turing85 opened this issue Oct 19, 2023 · 9 comments
Labels
area/jdbc Issues related to the JDBC extensions area/native-image kind/enhancement New feature or request

Comments

@turing85
Copy link
Contributor

Description

As of now, quarkus does not support Active Directory Authentication in native mode with the MS SQL driver. Since this is a standard authentication method for databases hosted in azure, it would be nice if quarkus supported AD authentication in native mode.


Story:
As a quarkus developer, developing a native image application
When I connect to a MS SQL database
Then I want the possibility to use AD authentication

Implementation ideas

No response

@turing85 turing85 added the kind/enhancement New feature or request label Oct 19, 2023
@gastaldi
Copy link
Contributor

@majguo @agoncal can anyone in Microsoft help with this?

@majguo
Copy link

majguo commented Oct 30, 2023

@gastaldi I created quarkiverse/quarkus-azure-services#169 by copying this issue, will raise it to the team in backlog grooming meeting next Monday.

@yrodiere
Copy link
Member

yrodiere commented Sep 5, 2024

Hello. Does this not duplicate #34580 ?

@maxandersen
Copy link
Member

maxandersen commented Sep 5, 2024

good context at #34580 (comment)
worth noting this did work in older versions (ie. 2.10) but mssql driver changed and we was more proactive in not adding optional dependencies.

some context found by @yrodiere:

Yoann Rodière: @maxandersen I did not have any contact with that team, no. Sanne might, but he won't be around for a few months.
I think talking to MS people is your best option, yes.

That being said, this is an rather old bug/requested feature:

#28913
#34580

Sanne explained the "why" here in particular: #34580 (comment)

Essentially, there's no blocker in principle. It's just that the MS SQL driver has an unreasonably large dependency tree, so we had to trim it. Adding active directory auth back will only be a matter of:

Testing. Got an MSSQL DB behind azure auth somewhere we can use? Especially on CI?
Removing a few exclusions.
Dealing with the resulting crashes -- I suppose some metadata is needed for native.
Yoann Rodière: BTW it seems that auth relies on an optional dependency, so we'll need to think if we want that dependency to be there by default in the Quarkus extension, or if we want to ask users to add it explicitly (and document this)

@yrodiere yrodiere added the area/jdbc Issues related to the JDBC extensions label Sep 5, 2024
@turing85
Copy link
Contributor Author

turing85 commented Sep 7, 2024

Hello. Does this not duplicate #34580 ?

No. #34580 is a general bug report. The bug has a workaround in JVM mode. This issue here is about making it work with native-image.

@turing85
Copy link
Contributor Author

turing85 commented Sep 7, 2024

Testing. Got an MSSQL DB behind azure auth somewhere we can use? Especially on CI?

If this was directed at me: no, I don't. I can ask around. But if I get an instance:

  • access will most probably be extremely limited, and
  • it will definitively not be permanent, i.e. the instance will most probably disappear after some time (maybe 1-2 months). So no permanenty solution for CI.

@yrodiere
Copy link
Member

yrodiere commented Sep 10, 2024

Testing. Got an MSSQL DB behind azure auth somewhere we can use? Especially on CI?

If this was directed at me: no, I don't.

It was directed at @maxandersen , who was a bit too enthusiastic in his copy-pasting :)

@Krabbee
Copy link

Krabbee commented Dec 2, 2024

Hello. Does this not duplicate #34580 ?

No. #34580 is a general bug report. The bug has a workaround in JVM mode. This issue here is about making it work with native-image.

Hi,
Can you hint to where I can find the work around in JVM mode?
I am trying to connect to a Azure SQL database from a Quarkus application.

@turing85
Copy link
Contributor Author

Hello. Does this not duplicate #34580 ?

No. #34580 is a general bug report. The bug has a workaround in JVM mode. This issue here is about making it work with native-image.

Hi, Can you hint to where I can find the work around in JVM mode? I am trying to connect to a Azure SQL database from a Quarkus application.

#34580 (comment) did the trick for us.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/jdbc Issues related to the JDBC extensions area/native-image kind/enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

6 participants