Closed hossam-nasr closed 3 months ago
Hmm, I agree that it should be its own file given now large it is.
Hossam also mentioned offline that it would be better to let users pass in the tenant id, service principal id, and the secret via parameters so they don't have to necessarily rely on the same exact environment variables that we are.
I agree with that idea since I'm sure people would like the option of being able to hard code the values for initial testing, acquire the values somewhere else, etc.
If it's going to be a separate file, where do you think it should live?
It can live in the root of this package.
Updated this to match the new constructor + moved the guide to a separate file + added more screenshots for easier guidance and readability :)
These screenshots are 🔥
These screenshots are 🔥
Yeah was going to say this too. This is really great. I'm only a tiny bit worried about keeping this up-to-date, but 🤷
@bwateratmsft Good call out, done!
@nturinski @alexweininger @MicroFish91 This actually dismissed y'all's approval rip, can you approve again? <3
[auth package] modify the README to add documentation of the newly created
AzureDevOpsSubscriptionProvider
, and include a guide on how to set up workflow identity federation on ADO to leverage that newly created class.