You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@drill.apache.org by GitBox <gi...@apache.org> on 2022/03/02 21:30:45 UTC

[GitHub] [drill] cgivre opened a new pull request #2479: DRILL-8153: Convert OAuth REST APIs to JSON

cgivre opened a new pull request #2479:
URL: https://github.com/apache/drill/pull/2479


   # [DRILL-8153](https://issues.apache.org/jira/browse/DRILL-8153): Convert OAuth REST APIs to JSON
   
   ## Description
   This PR converts the OAuth REST APIs to accept JSON.  
   
   ## Documentation
   The following API calls are now available for updating an HTTP Storage Plugin's OAuth tokens if OAuth is used. These three new endpoints are POST requests and accept the tokens as part of the form body.
   
   * `/storage/{plugin name}/update_access_token`
   * `/storage/{plugin name}/update_refresh_token`
   * `/storage/{plugin name}/update_oauth_tokens`
   
   These endpoints take as arguments: `accessToken`, `refreshToken` or both.
   
   ## Testing
   Updated unit tests. 


-- 
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: dev-unsubscribe@drill.apache.org

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



[GitHub] [drill] jnturton commented on pull request #2479: DRILL-8153: Convert OAuth REST APIs to JSON

Posted by GitBox <gi...@apache.org>.
jnturton commented on pull request #2479:
URL: https://github.com/apache/drill/pull/2479#issuecomment-1057680487


   @cgivre @tswagger please obtain a review from a Drill committer on PRs to this repo, as required by process :/.  Additional reviews from outside the public are welcome in addition, but not as a replacement.


-- 
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: dev-unsubscribe@drill.apache.org

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



[GitHub] [drill] cgivre merged pull request #2479: DRILL-8153: Convert OAuth REST APIs to JSON

Posted by GitBox <gi...@apache.org>.
cgivre merged pull request #2479:
URL: https://github.com/apache/drill/pull/2479


   


-- 
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: dev-unsubscribe@drill.apache.org

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



[GitHub] [drill] cgivre commented on pull request #2479: DRILL-8153: Convert OAuth REST APIs to JSON

Posted by GitBox <gi...@apache.org>.
cgivre commented on pull request #2479:
URL: https://github.com/apache/drill/pull/2479#issuecomment-1057463539


   Thanks @tswagger for the review!


-- 
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: dev-unsubscribe@drill.apache.org

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



[GitHub] [drill] jnturton edited a comment on pull request #2479: DRILL-8153: Convert OAuth REST APIs to JSON

Posted by GitBox <gi...@apache.org>.
jnturton edited a comment on pull request #2479:
URL: https://github.com/apache/drill/pull/2479#issuecomment-1057680487


   @cgivre @tswagger please obtain a review from a Drill committer on PRs to this repo, as required by process :/.  Additional reviews from outside the project are welcome in addition, but not as a replacement.


-- 
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: dev-unsubscribe@drill.apache.org

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