Re: [PR] Deprecate `oauth/tokens` endpoint [iceberg]

[via GitHub]

Fokko commented on code in PR #10603:
URL: https://github.com/apache/iceberg/pull/10603#discussion_r1670447305


##########
open-api/rest-catalog-open-api.yaml:
##########
@@ -134,9 +134,17 @@ paths:
     post:
       tags:
         - OAuth2 API
-      summary: Get a token using an OAuth2 flow
+      summary: Get a token using an OAuth2 flow (DEPRECATED for REMOVAL)
+      deprecated: true
       operationId: getToken
       description:
+        The `oauth/tokens` endpoint is **DEPRECATED for REMOVAL**. It is _not_ 
recommended to

Review Comment:
   I think Daniel is pointing at when the endpoint will go:
   
   
![image](https://github.com/apache/iceberg/assets/1134248/395ca590-0b61-4fe3-b9a9-8cacd19b08e5)
   
   So that should be 1.8 or 2.0 according to the proposal, where it is cleaner 
to remove it at 2.0. 



-- 
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: issues-unsubscr...@iceberg.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@iceberg.apache.org
For additional commands, e-mail: issues-h...@iceberg.apache.org