search

AzureAD / azure-activedirectory-library-for-js

The code for ADAL.js and ADAL Angular has been moved to the MSAL.js repo. Please open any issues or PRs at the link below.
https://github.com/AzureAD/microsoft-authentication-library-for-js/tree/dev/maintenance/adal-angular
Apache License 2.0
627 stars 372 forks source link

Redirect URI should end with / #769

Closed MarkHerhold closed 6 years ago

MarkHerhold commented 6 years ago

Per this issue, ADAL should probably help users out by adding a trailing / or error out when one is not provided in the redirect URI.

asnow003 commented 6 years ago

@MarkHerhold I just did a generic test on my mac using the latest Safari in Sierra. I tested using one of our basic samples and didn't see an error. Could you provide your error or a sample that demonstrates the issue?

MarkHerhold commented 6 years ago

I used the same environment. I must be wrong or there must also be some other factor at play.