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

Added OAuthBearer authentication mode #437

Merged
merged 4 commits into from
Jul 25, 2024
Merged

Added OAuthBearer authentication mode #437

merged 4 commits into from
Jul 25, 2024

Conversation

nakah
Copy link
Contributor

@nakah nakah commented Jun 13, 2023

Adds OAuthBearer authentication mode support as requested in #427.
Had to bump Confluent.Kafka to 1.9.3 because there was a bug in 1.9.0 during authentication with identity provider.

I've tested using a new Confluent Kafka cluster deployed on Azure using Azure AD as idendity provider.

@nakah
Copy link
Contributor Author

nakah commented Jun 13, 2023

@microsoft-github-policy-service agree

@steph409
Copy link

steph409 commented Feb 8, 2024

any updates on this? I would really need this feature!

@penicaudm
Copy link

We would also be very interested in OAuth support for Confluent Kafka trigger/publish output bindings.

@Quenik
Copy link

Quenik commented Mar 18, 2024

It would be great to see this feature in production asap! Confluent.Kafka supports this authentication mode already so imho there's no big deal 'bout it. It would have a huge impact for us.

@jainharsh98 jainharsh98 merged commit bf8ffe6 into Azure:dev Jul 25, 2024
1 check passed
jainharsh98 added a commit that referenced this pull request Jul 29, 2024
Co-authored-by: Hakan Altinbasak <hakan.altinbasak@avanade.com>
Co-authored-by: jainharsh98 <jainh@microsoft.com>
jainharsh98 added a commit that referenced this pull request Jul 30, 2024
* Changing all references to net8.0 and build errors

* Fixing end to end tests for confluent

* Changes to make Lang E22E tests succeed

* Install .net8 sdk in the build pipeline

* removing confluent sdk dependency on e2e tests

* Upgrading confluent package to 2.4.0(#504)

* Fix scale monitor descriptor function id (#451)

Co-authored-by: jainharsh98 <jainh@microsoft.com>

* SslKeyPassword resolution as secret setting for output binding (#464)

Co-authored-by: Gregory van de Wiele <Gregory.vandeWiele@eneco.com>
Co-authored-by: jainharsh98 <jainh@microsoft.com>

* Added OAuthBearer authentication mode (#437)

Co-authored-by: Hakan Altinbasak <hakan.altinbasak@avanade.com>
Co-authored-by: jainharsh98 <jainh@microsoft.com>

* Fixing Msg commit for Cancellation Scenarios (#505)

* Commiting msg in single mode if cancellation is not requested.

* Removing stray space character

* Removing checks for specific and generic records to allow for other data types for out of proc (#506)

* Fix for schema registry and UTs (#507)

* Updating extension version to 4.0.0

* Resolving Oauthbearer Authentication Attributes from Settings for Output Binding (#511)

* Resolving secure settings from settings for oauthbearer authentication

* Adding Unit Test cases for oauthbearer settings

---------

Co-authored-by: Vivek Jilla <vijilla@microsoft.com>
Co-authored-by: Vivek Jilla <vivekjilla7@gmail.com>
Co-authored-by: Pierre Sk <17861564+pierre-sk@users.noreply.github.com>
Co-authored-by: gvdwiele <22445+gvdwiele@users.noreply.github.com>
Co-authored-by: Gregory van de Wiele <Gregory.vandeWiele@eneco.com>
Co-authored-by: Hakan <hakan.altinbasak@gmail.com>
Co-authored-by: Hakan Altinbasak <hakan.altinbasak@avanade.com>
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.

5 participants