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

    https://github.com/apache/nifi/pull/1462#discussion_r99034384
  
    --- Diff: 
nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/GetHTTP.java
 ---
    @@ -102,6 +96,11 @@
         + "once the content has been fetched from the given URL, it will not 
be fetched again until the content on the remote server changes. Note that due 
to limitations on state "
         + "management, stored \"last modified\" and etag fields never expire. 
If the URL in GetHttp uses Expression Language that is unbounded, there "
         + "is the potential for Out of Memory Errors to occur.")
    +@DynamicProperties({
    +    @DynamicProperty(name = "header_XYZ", value = "The Expression Language 
to be used to populate the header value", description = "The additional headers 
to be sent by the processor " +
    --- End diff --
    
    This is a little confusing as it seems like the prefix header_ is needed 
before the name of the actual header but that doesn't seem to be the 
implementation.  What are your thoughts?


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to