Skip to content

Commit

Permalink
Remove trailing slash as Url parsing code doesn't expect it. (#334)
Browse files Browse the repository at this point in the history
* Remove trailing slash as Url parsing code doesn't expect it.

* add @ symbol to email lookup
  • Loading branch information
JoshLove-msft authored Jan 17, 2020
1 parent c0d6ce2 commit b17d295
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/dotnet/APIView/APIViewWeb/Startup.cs
Original file line number Diff line number Diff line change
Expand Up @@ -181,7 +181,7 @@ private static async Task<string> GetMicrosoftEmailAsync(OAuthCreatingTicketCont
foreach (var email in emails)
{
var address = email["email"]?.Value<string>();
if (address != null && address.EndsWith("microsoft.com", StringComparison.OrdinalIgnoreCase))
if (address != null && address.EndsWith("@microsoft.com", StringComparison.OrdinalIgnoreCase))
{
return address;
}
Expand Down
2 changes: 1 addition & 1 deletion src/dotnet/APIView/APIViewWeb/appsettings.json
Original file line number Diff line number Diff line change
Expand Up @@ -8,5 +8,5 @@
"Github": {
"RequiredOrganization": [ "Azure", "Microsoft" ]
},
"EndPoint": "https://apiview.dev/"
"EndPoint": "https://apiview.dev"
}

0 comments on commit b17d295

Please sign in to comment.