lapchynski commented on pull request #517: URL: https://github.com/apache/guacamole-client/pull/517#issuecomment-642137980
@knacktim It looks like at least AWS Cognito and GCP Identity Platform have a discovery document endpoint, although for Cognito it doesn't seem to be documented or I couldn't find documentation for it if it is. I think using the discovery document for validation would be a good idea, but since if we're already using that data for validation why not just set the rest of the options (authorization-endpoint, jwks-endpoint, issuer, etc) directly from it and make the equivalent guacamole.properties options optional overrides? If I'm understanding the spec correctly, that seems like it's kinda the point of the discovery document to begin with. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
