Updating AspNetCore sample token handling based on updated security g…

…uidance
pnp · Mar 27, 2020 · a746e63 · a746e63
1 parent 60acbec
commit a746e63
Showing 1 changed file with 21 additions and 0 deletions.
diff --git a/...tion/src/SharePointPnP.IdentityModel/Extensions/S2S/Tokens/JsonWebSecurityTokenHandler.cs b/...tion/src/SharePointPnP.IdentityModel/Extensions/S2S/Tokens/JsonWebSecurityTokenHandler.cs
@@ -1,4 +1,6 @@
 using Microsoft.IdentityModel.Claims;
+using System;
+using System.Linq;
 using System.ServiceModel.Security.Tokens;
 using System.Xml;
 
@@ -412,6 +414,7 @@ private ClaimsIdentityCollection ValidateTokenCore(System.IdentityModel.Tokens.S
             ClaimsIdentity claimsIdentity = new ClaimsIdentity("Federation");
             if (!isActorToken && jsonWebSecurityToken.ActorToken != null)
             {
+                ValidateActorTokenForAppOnly(jsonWebSecurityToken.ActorToken);
                 ClaimsIdentityCollection claimsIdentityCollection2 = this.ValidateActorToken(jsonWebSecurityToken.ActorToken);
                 if (claimsIdentityCollection2.Count > 1)
                 {
@@ -440,6 +443,24 @@ private ClaimsIdentityCollection ValidateTokenCore(System.IdentityModel.Tokens.S
             return claimsIdentityCollection;
         }
 
+        /// <summary>
+        ///Validates that the actor token is an app token by checking for the lack of user claims
+        /// </summary>
+        /// <param name="actorToken"></param>
+        private static void ValidateActorTokenForAppOnly(JsonWebSecurityToken actorToken)
+        {
+            if (actorToken != null)
+            {
+                if (actorToken.Claims.FirstOrDefault<JsonWebTokenClaim>(x => x.ClaimType.Equals("scp")) != null
+                  || actorToken.Claims.FirstOrDefault<JsonWebTokenClaim>(x => x.ClaimType.Equals("upn")) != null
+                  || actorToken.Claims.FirstOrDefault<JsonWebTokenClaim>(x => x.ClaimType.Equals("unique_name")) != null
+                  || actorToken.Claims.FirstOrDefault<JsonWebTokenClaim>(x => x.ClaimType.Equals("altsecid")) != null)
+                {
+                    throw new UnauthorizedAccessException("Invalid actor token.");
+                }
+            }
+        }
+
         public override ClaimsIdentityCollection ValidateToken(System.IdentityModel.Tokens.SecurityToken token)
         {
             return this.ValidateTokenCore(token, false);