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

    https://github.com/apache/nifi/pull/2723#discussion_r192722942
  
    --- Diff: 
nifi-nar-bundles/nifi-standard-services/nifi-lookup-services-bundle/nifi-lookup-services/src/main/java/org/apache/nifi/lookup/RestLookupService.java
 ---
    @@ -277,6 +280,20 @@ private void setProxy(OkHttpClient.Builder builder) {
             }
         }
     
    --- End diff --
    
    I don't think so because the EL support is enabled on the fly by the user, 
and any user who is enterprising enough to do that should be responsible for 
validating it. There's no good fallback option short of having them specify yet 
another key, and IMO that's overkill.


---

Reply via email to