Github user alopresto commented on a diff in the pull request:

    https://github.com/apache/nifi/pull/3020#discussion_r219712932
  
    --- Diff: 
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/InvokeHTTP.java
 ---
    @@ -714,11 +721,11 @@ private void setSslSocketFactory(OkHttpClient.Builder 
okHttpClientBuilder, SSLCo
         }
     
         private void setAuthenticator(OkHttpClient.Builder 
okHttpClientBuilder, ProcessContext context) {
    -        final String authUser = 
trimToEmpty(context.getProperty(PROP_BASIC_AUTH_USERNAME).getValue());
    +        final String authUser = 
trimToEmpty(context.getProperty(PROP_BASIC_AUTH_USERNAME).evaluateAttributeExpressions().getValue());
     
             // If the username/password properties are set then check if 
digest auth is being used
             if (!authUser.isEmpty() && 
"true".equalsIgnoreCase(context.getProperty(PROP_DIGEST_AUTH).getValue())) {
    -            final String authPass = 
trimToEmpty(context.getProperty(PROP_BASIC_AUTH_PASSWORD).getValue());
    +            final String authPass = 
trimToEmpty(context.getProperty(PROP_BASIC_AUTH_PASSWORD).evaluateAttributeExpressions().getValue());
    --- End diff --
    
    See comment about `password` above. 


---

Reply via email to