add comment

erwinkramer · erwinkramer · commit d4ce453814a4 · 2026-01-26T16:25:32.000+01:00
diff --git a/BankApi.Mcp/Builder.MCP.cs b/BankApi.Mcp/Builder.MCP.cs
@@ -17,6 +17,8 @@ public static async Task<IServiceCollection> AddMCPService(this IServiceCollecti
 
         services.AddScoped(sp =>
         {
+            // This shouldn't be needed, since proper MCP clients handle this via the "/.well-known/oauth-protected-resource" endpoint
+            // Keep this in until following issue is solved: https://github.com/abdebek/MCPify/issues/20
             return new OAuthAuthorizationCodeAuthentication(
                 clientId: "b6997777-3799-4c55-b78a-4ce96e3d959c",
                 authorizationEndpoint: $"https://login.microsoftonline.com/{tenantId}/oauth2/v2.0/authorize",

Original file line number	Diff line number	Diff line change
`@@ -17,6 +17,8 @@ public static async Task<IServiceCollection> AddMCPService(this IServiceCollecti`
`17`	`17`
`18`	`18`	`services.AddScoped(sp =>`
`19`	`19`	`{`
	`20`	`+ // This shouldn't be needed, since proper MCP clients handle this via the "/.well-known/oauth-protected-resource" endpoint`
	`21`	`+ // Keep this in until following issue is solved: https://github.com/abdebek/MCPify/issues/20`
`20`	`22`	`return new OAuthAuthorizationCodeAuthentication(`
`21`	`23`	`clientId: "b6997777-3799-4c55-b78a-4ce96e3d959c",`
`22`	`24`	`authorizationEndpoint: $"https://login.microsoftonline.com/{tenantId}/oauth2/v2.0/authorize",`