FANNG1 commented on code in PR #7434:
URL: https://github.com/apache/gravitino/pull/7434#discussion_r2174860915


##########
docs/iceberg-rest-service.md:
##########
@@ -81,7 +81,63 @@ You can also specify filter parameters by setting 
configuration entries in the s
 
 ### Security
 
-Gravitino Iceberg REST server supports OAuth2 and HTTPS, please refer to 
[Security](security/security.md) for more details.
+#### OAuth2
+
+Please refer to [OAuth2 
Configuration](./security/how-to-authenticate#server-configuration) for how to 
enable OAuth2.
+
+When enabling OAuth2 and leveraging a dynamic configuration provider to 
retrieve catalog information from the Gravitino server, please use the 
following configuration parameters to establish OAuth2 authentication for 
secure communication with the Gravitino server:

Review Comment:
   Most Iceberg REST server configuration items are seperated by `-`, I prefer 
to keep consistent with current style WDYT?



-- 
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]

Reply via email to