Skip to content
This repository has been archived by the owner on Dec 13, 2018. It is now read-only.

added redirectUri parameter to GetAuthorizationUrl #56

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

added redirectUri parameter to GetAuthorizationUrl #56

wants to merge 1 commit into from

Conversation

bcwood
Copy link

@bcwood bcwood commented Oct 16, 2014

By specifying the return_url parameter to /authorize, the user will automatically be redirected back to the calling web application after authorizing access.

Instead of creating an overloaded method for this, I thought about including redirectUri as an optional parameter, but in the original implementation of GetAuthorizationUrl, you're using /connect instead of /authorize, but I couldn't find any documentation on /connect. Perhaps this is an old endpoint that is only left open for backwards compatibility?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant