microsoft / microsoft/azure-devops-auth-samples
Sample MangedClientConsoleAppSample Doesn't Work (on branch master)
Nobody has claimed this yet.
- Dominant language
- JavaScript
- Stars
- 277
- Forks
- 307
- Avg merge
- 3d 18h
- Merged PRs (30d)
- 2
Description
It appears there is an incomplete attempt to port it to .NET Core but that won't work ([interactive isn't supported yet on .NET core)[https://github.com/AzureAD/azure-activedirectory-library-for-dotnet/issues/879].
To get it to work I needed to:
- Remove the
netcoreapp2.0target - Include package
Microsoft.VisualStudio.Services.InteractiveClient(despite step 3 of the readme saying it was in place) - Remove the parameterless constructor call for
PlatformParameters - Change the base URL to be
https.
(The last point took a while to find: getting unauthorised responses after authorising isn't the greatest diagnostic.)
Sample code really should be ready to run!
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Inspect the ManagedClientConsoleAppSample and its README first, comparing the current setup with the four changes listed in the issue. The work is done when the sample builds and runs successfully on the master branch, including interactive authorization without unauthorized responses.
Written by the indexing model from the issue text.
Assessment
- Domain
- authentication
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100