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

go-redis/redis - Microsoft Entra Auth Sample #17384

Open
joshfree opened this issue Mar 25, 2022 · 3 comments
Open

go-redis/redis - Microsoft Entra Auth Sample #17384

joshfree opened this issue Mar 25, 2022 · 3 comments
Assignees
Labels
Azure.Identity Client This issue points to a problem in the data-plane of the library.
Milestone

Comments

@joshfree
Copy link
Member

Tracking issue for dogfooding and creating code samples (or documentation) to show best practices for how to use AAD Auth with popular 3rd party OSS Redis clients when connecting to Azure.

Depending on the outcome of the dogfooding:

Possible Outcome 1: 0-1 days of extra work (minimal sample code is required to enable the experience):

  • Check-in a thin / simple code sample project to our sdk repo.
  • Share the sample with the Azure Redis team so that they can then share the sample with customers, publish updated docs, and also validate the service handles credential revocation and similar scenarios to test with the provided sample.

Possible Outcome 2: ~1 week of extra work (significant sample code is required to enable the experience):

  • Discuss the investigation results with the Identity SDK and Redis teams before proceeding
  • Based on the discussion, decide on whether we should be building a sample or an extension library that is shipped instead.

Possible Outcome 3: (using AAD Auth with the client is blocked until code changes are first made to the OSS Redis client):

  • Discuss the investigation results with the Identity SDK and Redis teams.
  • File GitHub issue with 3rd party project. Provide as much details as possible, including any code samples, gists, etc.
@joshfree joshfree added Client This issue points to a problem in the data-plane of the library. Azure.Identity labels Mar 25, 2022
@joshfree joshfree added this to the Backlog milestone Mar 25, 2022
Copy link

Hi @joshfree, we deeply appreciate your input into this project. Regrettably, this issue has remained inactive for over 2 years, leading us to the decision to close it. We've implemented this policy to maintain the relevance of our issue queue and facilitate easier navigation for new contributors. If you still believe this topic requires attention, please feel free to create a new issue, referencing this one. Thank you for your understanding and ongoing support.

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Mar 25, 2024
@github-actions github-actions bot locked and limited conversation to collaborators Mar 25, 2024
@scottaddie
Copy link
Member

go-redis v9.5.2 added an Options.CredentialsProviderContext feature that supports Microsoft Entra authN. Let's add a code sample to this repo. Once completed, add a new table row to this Learn doc section with a link to the sample.

@scottaddie scottaddie reopened this Jul 9, 2024
@scottaddie scottaddie changed the title go-redis/redis - AAD Auth Sample go-redis/redis - Microsoft Entra Auth Sample Jul 9, 2024
@scottaddie scottaddie modified the milestones: Backlog, 2024-08 Jul 9, 2024
@chlowell chlowell modified the milestones: 2024-08, 2024-10 Aug 2, 2024
@chlowell
Copy link
Member

The example is live in the package docs for v1.8.0-beta.2. I'll leave this issue open to track updating the Learn doc after the stable v1.8.0 release.

@chlowell chlowell modified the milestones: 2024-10, 2024-11 Sep 23, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Azure.Identity Client This issue points to a problem in the data-plane of the library.
Projects
Status: In Progress
Development

No branches or pull requests

3 participants