Skip to content

Commit 5a29941

Browse files
xperiandrikevinchalet
authored andcommitted
Fix the LinkedIn provider to invoke the CreatingTicket event and switch to the new LinkedIn API
1 parent 1909b75 commit 5a29941

File tree

13 files changed

+5
-13
lines changed

13 files changed

+5
-13
lines changed

src/AspNet.Security.OAuth.Bitbucket/BitbucketAuthenticationHandler.cs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,6 @@ protected override async Task<AuthenticationTicket> CreateTicketAsync([NotNull]
6868
}
6969

7070
await Options.Events.CreatingTicket(context);
71-
7271
return new AuthenticationTicket(context.Principal, context.Properties, Scheme.Name);
7372
}
7473

src/AspNet.Security.OAuth.Buffer/BufferAuthenticationHandler.cs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,6 @@ protected override async Task<AuthenticationTicket> CreateTicketAsync([NotNull]
5555
context.RunClaimActions(payload);
5656

5757
await Options.Events.CreatingTicket(context);
58-
5958
return new AuthenticationTicket(context.Principal, context.Properties, Scheme.Name);
6059
}
6160
}

src/AspNet.Security.OAuth.CiscoSpark/CiscoSparkAuthenticationHandler.cs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,7 +56,6 @@ protected override async Task<AuthenticationTicket> CreateTicketAsync([NotNull]
5656
context.RunClaimActions(payload);
5757

5858
await Options.Events.CreatingTicket(context);
59-
6059
return new AuthenticationTicket(context.Principal, context.Properties, Scheme.Name);
6160
}
6261
}

src/AspNet.Security.OAuth.DeviantArt/DeviantArtAuthenticationHandler.cs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,6 @@ protected override async Task<AuthenticationTicket> CreateTicketAsync([NotNull]
5555
context.RunClaimActions(payload);
5656

5757
await Options.Events.CreatingTicket(context);
58-
5958
return new AuthenticationTicket(context.Principal, context.Properties, Scheme.Name);
6059
}
6160
}

src/AspNet.Security.OAuth.EVEOnline/EVEOnlineAuthenticationHandler.cs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,6 @@ protected override async Task<AuthenticationTicket> CreateTicketAsync([NotNull]
5555
context.RunClaimActions(payload);
5656

5757
await Options.Events.CreatingTicket(context);
58-
5958
return new AuthenticationTicket(context.Principal, context.Properties, Scheme.Name);
6059
}
6160
}

src/AspNet.Security.OAuth.LinkedIn/LinkedInAuthenticationDefaults.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -37,12 +37,12 @@ public class LinkedInAuthenticationDefaults
3737
/// <summary>
3838
/// Default value for <see cref="OAuthOptions.AuthorizationEndpoint"/>.
3939
/// </summary>
40-
public const string AuthorizationEndpoint = "https://www.linkedin.com/uas/oauth2/authorization";
40+
public const string AuthorizationEndpoint = "https://www.linkedin.com/oauth/v2/authorization";
4141

4242
/// <summary>
4343
/// Default value for <see cref="OAuthOptions.TokenEndpoint"/>.
4444
/// </summary>
45-
public const string TokenEndpoint = "https://www.linkedin.com/uas/oauth2/accessToken";
45+
public const string TokenEndpoint = "https://www.linkedin.com/oauth/v2/accessToken";
4646

4747
/// <summary>
4848
/// Default value for <see cref="OAuthOptions.UserInformationEndpoint"/>.

src/AspNet.Security.OAuth.LinkedIn/LinkedInAuthenticationHandler.cs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -62,6 +62,7 @@ protected override async Task<AuthenticationTicket> CreateTicketAsync([NotNull]
6262
var context = new OAuthCreatingTicketContext(principal, properties, Context, Scheme, Options, Backchannel, tokens, payload);
6363
context.RunClaimActions(payload);
6464

65+
await Options.Events.CreatingTicket(context);
6566
return new AuthenticationTicket(context.Principal, context.Properties, Scheme.Name);
6667
}
6768
}

src/AspNet.Security.OAuth.QQ/QQAuthenticationHandler.cs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -82,7 +82,6 @@ protected override async Task<AuthenticationTicket> CreateTicketAsync(
8282
context.RunClaimActions(payload);
8383

8484
await Options.Events.CreatingTicket(context);
85-
8685
return new AuthenticationTicket(context.Principal, context.Properties, Scheme.Name);
8786
}
8887

src/AspNet.Security.OAuth.Reddit/RedditAuthenticationHandler.cs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,6 @@ protected override async Task<AuthenticationTicket> CreateTicketAsync([NotNull]
6666
context.RunClaimActions(payload);
6767

6868
await Options.Events.CreatingTicket(context);
69-
7069
return new AuthenticationTicket(context.Principal, context.Properties, Scheme.Name);
7170
}
7271

src/AspNet.Security.OAuth.SoundCloud/SoundCloudAuthenticationHandler.cs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,6 @@ protected override async Task<AuthenticationTicket> CreateTicketAsync([NotNull]
5757
context.RunClaimActions(payload);
5858

5959
await Options.Events.CreatingTicket(context);
60-
6160
return new AuthenticationTicket(context.Principal, context.Properties, Scheme.Name);
6261
}
6362
}

0 commit comments

Comments
 (0)