-
Notifications
You must be signed in to change notification settings - Fork 301
Add "envvar" authentication option #661
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 "envvar" authentication option #661
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #661 +/- ##
=======================================
Coverage ? 96.47%
=======================================
Files ? 16
Lines ? 1249
Branches ? 246
=======================================
Hits ? 1205
Misses ? 17
Partials ? 27
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
Sorry @kyle-rader-msft but we need folks to follow the process outlined in https://github.com/microsoft/azure-devops-mcp/blob/main/CONTRIBUTING.md Please create an Issue first before creating a PR. I will going to close this PR for the time being. |
|
@aaudzei assigning to you to review per Teams chat |
22947c6 to
9c4dce0
Compare
f43d029 to
d2e6fd3
Compare
Support passing in a token via environment variable
#608
Associated Risks
There are very few risks here, as taking in an EntraID token inherently means another tool has already handled authentication and we now have a short-lived token.
✅ PR Checklist
🧪 How did you test it?
I've manually tested by