[asterixdb] branch master updated: [NO ISSUE][*DB] Refactor UDF auth header access

[mblow]

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

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


The following commit(s) were added to refs/heads/master by this push:
     new f4380e5d48 [NO ISSUE][*DB] Refactor UDF auth header access
f4380e5d48 is described below

commit f4380e5d489c6f699ef14b8f47d35f20d8a71fb6
Author: Michael Blow <michael.b...@couchbase.com>
AuthorDate: Sun Feb 19 12:50:16 2023 -0500

    [NO ISSUE][*DB] Refactor UDF auth header access
    
    Change-Id: I41023663e1212c0c8789873e63c9bfbeae48cea8
    Reviewed-on: https://asterix-gerrit.ics.uci.edu/c/asterixdb/+/17390
    Integration-Tests: Jenkins <jenk...@fulliautomatix.ics.uci.edu>
    Tested-by: Jenkins <jenk...@fulliautomatix.ics.uci.edu>
    Reviewed-by: Murtadha Hubail <mhub...@apache.org>
---
 .../java/org/apache/asterix/api/http/server/NCUdfApiServlet.java  | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git 
a/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/NCUdfApiServlet.java
 
b/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/NCUdfApiServlet.java
index 3d0f7fcbd8..47685d8cce 100644
--- 
a/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/NCUdfApiServlet.java
+++ 
b/asterixdb/asterix-app/src/main/java/org/apache/asterix/api/http/server/NCUdfApiServlet.java
@@ -81,7 +81,7 @@ public class NCUdfApiServlet extends AbstractNCUdfServlet {
     protected final IReceptionist receptionist;
 
     protected Path workingDir;
-    protected String sysAuthHeader;
+    private String sysAuthHeader;
     private ILibraryManager libraryManager;
     private int timeout;
 
@@ -250,7 +250,7 @@ public class NCUdfApiServlet extends AbstractNCUdfServlet {
                 }
                 URI downloadURI = createDownloadURI(libraryTempFile);
                 doCreate(dvAndName.getFirst(), dvAndName.getSecond(), language,
-                        ExternalLibraryUtils.digestToHexString(digest), 
downloadURI, true, sysAuthHeader,
+                        ExternalLibraryUtils.digestToHexString(digest), 
downloadURI, true, getSysAuthHeader(),
                         requestReference, request);
             } else if (op == LibraryOperation.DELETE) {
                 //DELETE semantics imply ifExists
@@ -281,6 +281,10 @@ public class NCUdfApiServlet extends AbstractNCUdfServlet {
         }
     }
 
+    protected String getSysAuthHeader() {
+        return sysAuthHeader;
+    }
+
     private void writeException(Exception e, IServletResponse response) {
         response.setStatus(toHttpErrorStatus(e));
         PrintWriter responseWriter = response.writer();