-
Notifications
You must be signed in to change notification settings - Fork 109
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
Struggling with V2 #83
Comments
What exactly is the issue? |
Sorry I hit the enter button too quickly and saved this before I had typed anything. I am typing up the issue, give me just a minute. |
I am new to OAuth2, Azure, and Active directory, but I have been asked to work on an integration. After reading this issue #82 it seemed like starting with V2 of this library was the best place to start. I am basing most of my code off of the example code in the readme, but either I am doing something terribly wrong or the example is not complete. So far here is what I have
However I was getting errors from guzzle
because this is all new to me I am not sure if this is correct or not. Or if I am using the library incorrectly. I also accept that I may have setup AD in Azure incorrectly. Any tips on where to start would be much appreciated. |
You are on the right path with The issue is that https://github.com/TheNetworg/oauth2-azure/blob/v2.0.0/src/Provider/Azure.php#L232 calls |
Sweet, that worked. I will work on a PR |
…m getResponse to getParsedResponse
I created a PR #84 |
I merged the PR, thanks for the contirbution! |
Awesome, thanks for helping me figure that out. It would be great to get the example in the V2 branch to work out of the box for the V2 microsoft API. For example, adding scope and removing resource, and changing the |
No description provided.
The text was updated successfully, but these errors were encountered: