fisjac commented on code in PR #30081:
URL: https://github.com/apache/superset/pull/30081#discussion_r1762220050
##########
superset/utils/oauth2.py:
##########
@@ -192,3 +192,8 @@ class OAuth2ClientConfigSchema(Schema):
)
authorization_request_uri = fields.String(required=True)
token_request_uri = fields.String(required=True)
+ request_content_type = fields.String(
Review Comment:
I might be mistaken, but I believe this validation is only applied to the
client_info contained within `encrypted_extra` when provided by a user? Is the
intent that this is where the `request_content_type` is going to be provided,
or is it going to be set as a default for the Trino engine spec?
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
