fix: fixed an inaccuracy in the protocol implementation

Author: KirillBorisovich

HwProj.APIGateway/HwProj.APIGateway.API/Lti/Controllers/LtiDeepLinkingReturnController.cs

@@ -42,7 +42,7 @@ public async Task<IActionResult> OnDeepLinkingReturnAsync([FromForm] IFormCollec
         }
 
         var unverifiedToken = Handler.ReadJwtToken(tokenString);
-        var clientId = unverifiedToken.Subject;
+        var clientId = unverifiedToken.Issuer;
 
         var tool = toolService.GetByClientId(clientId);
         if (tool == null)
@@ -58,7 +58,7 @@ public async Task<IActionResult> OnDeepLinkingReturnAsync([FromForm] IFormCollec
             Handler.ValidateToken(tokenString, new TokenValidationParameters
             {
                 ValidateIssuer = true,
-                ValidIssuer = tool.issuer,
+                ValidIssuer = tool.ClientId,
                 ValidateAudience = true,
                 ValidAudience = ltiPlatformOptions.Value.Issuer,
                 ValidateLifetime = true,

HwProj.APIGateway/HwProj.APIGateway.API/Lti/Controllers/MockToolController.cs

@@ -36,7 +36,7 @@ private record MockTask(string Id, string Title, string Description, int Score);
     static MockToolController()
     {
         var rsa = RSA.Create(2048);
-        var keyId = "mock-tool-key-id";
+        const string keyId = "mock-tool-key-id";
         SigningKey = new RsaSecurityKey(rsa) { KeyId = keyId };
     }
 
@@ -172,8 +172,7 @@ public IActionResult SubmitDeepLinkingSelection(
 
         var payload = new JwtPayload
         {
-            { "iss", ToolIss },
-            { "sub", ToolNameId },
+            { "iss", ToolNameId },
             { "aud", platformIssuer },
             { "iat", DateTimeOffset.UtcNow.ToUnixTimeSeconds() },
             { "exp", DateTimeOffset.UtcNow.AddMinutes(5).ToUnixTimeSeconds() },