This is an automated email from the ASF dual-hosted git repository.

dzamo pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/drill.git


The following commit(s) were added to refs/heads/master by this push:
     new 73a829a   DRILL-8126: Ignore OAuth Parameter in Storage Plugin
73a829a is described below

commit 73a829a5a0eb21fc35d6cfd878310b7069135ecd
Author: Charles S. Givre <[email protected]>
AuthorDate: Tue Feb 8 02:41:35 2022 -0500

     DRILL-8126: Ignore OAuth Parameter in Storage Plugin
---
 .../java/org/apache/drill/exec/server/rest/PluginConfigWrapper.java     | 2 ++
 1 file changed, 2 insertions(+)

diff --git 
a/exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/PluginConfigWrapper.java
 
b/exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/PluginConfigWrapper.java
index 397b264..6454a55 100644
--- 
a/exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/PluginConfigWrapper.java
+++ 
b/exec/java-exec/src/main/java/org/apache/drill/exec/server/rest/PluginConfigWrapper.java
@@ -19,6 +19,7 @@ package org.apache.drill.exec.server.rest;
 
 import javax.xml.bind.annotation.XmlRootElement;
 
+import com.fasterxml.jackson.annotation.JsonIgnore;
 import org.apache.commons.lang3.StringUtils;
 import org.apache.drill.common.logical.AbstractSecuredStoragePluginConfig;
 import org.apache.drill.common.logical.StoragePluginConfig;
@@ -63,6 +64,7 @@ public class PluginConfigWrapper {
    * 3. The credentialsProvider must contain a client_id and client_secret
    * @return true if the plugin uses OAuth, false if not.
    */
+  @JsonIgnore
   public boolean isOauth() {
     if (! (config instanceof AbstractSecuredStoragePluginConfig)) {
       return false;

Reply via email to