[ 
https://issues.apache.org/jira/browse/DRILL-8220?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17537467#comment-17537467
 ] 

ASF GitHub Bot commented on DRILL-8220:
---------------------------------------

jnturton commented on code in PR #2544:
URL: https://github.com/apache/drill/pull/2544#discussion_r873581651


##########
exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/StorageResources.java:
##########
@@ -212,61 +195,15 @@ public Response enablePlugin(@PathParam("name") String 
name, @PathParam("val") B
   @Consumes(MediaType.APPLICATION_JSON)
   @Produces(MediaType.APPLICATION_JSON)
   public Response updateRefreshToken(@PathParam("name") String name, 
OAuthTokenContainer tokens) {

Review Comment:
   Thanks, perhaps deprecation is better even though the next version is 2.0





> Add User Translation Support for OAuth Enabled Plugins
> ------------------------------------------------------
>
>                 Key: DRILL-8220
>                 URL: https://issues.apache.org/jira/browse/DRILL-8220
>             Project: Apache Drill
>          Issue Type: Improvement
>          Components: Storage - Other
>    Affects Versions: 1.20.0
>            Reporter: Charles Givre
>            Assignee: Charles Givre
>            Priority: Major
>             Fix For: 2.0.0
>
>
> This PR adds support for individual users to provide their own credentials 
> for plugins that use OAuth 2.0 as a means of authorization and 
> authentication.   Currently, only the HTTP storage plugin supports OAuth, 
> however, this PR moves some of the core features out of the HTTP plugin so 
> that other plugins can access this. 



--
This message was sent by Atlassian Jira
(v8.20.7#820007)

Reply via email to