[nifi] branch master updated: NIFI-7420 remove the http.param attribute. It contained both the query parameters already captured in http.query.param _and_ the multipart form data names and values, which are captured in the part data, and could be very very large

[markap14]

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

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


The following commit(s) were added to refs/heads/master by this push:
     new f22cd26  NIFI-7420 remove the http.param attribute. It contained both 
the query parameters already captured in http.query.param _and_ the multipart 
form data names and values, which are captured in the part data, and could be 
very very large
f22cd26 is described below

commit f22cd26cf41cdce95106a05f8be3ad41c8d89f45
Author: Otto Fowler <ottobackwa...@gmail.com>
AuthorDate: Fri May 1 18:37:25 2020 -0400

    NIFI-7420 remove the http.param attribute. It contained both the query 
parameters already captured in http.query.param _and_ the multipart form data 
names and values, which are captured in the part data, and could be very very 
large
    
    This closes #4251.
    
    Signed-off-by: Mark Payne <marka...@hotmail.com>
---
 .../org/apache/nifi/processors/standard/HandleHttpRequest.java   | 9 ++-------
 1 file changed, 2 insertions(+), 7 deletions(-)

diff --git 
a/nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/HandleHttpRequest.java
 
b/nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/HandleHttpRequest.java
index 66b940a..bdc9a99 100644
--- 
a/nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/HandleHttpRequest.java
+++ 
b/nifi-nar-bundles/nifi-standard-bundle/nifi-standard-processors/src/main/java/org/apache/nifi/processors/standard/HandleHttpRequest.java
@@ -112,6 +112,8 @@ import java.util.regex.Pattern;
     @WritesAttribute(attribute = HTTPUtils.HTTP_REQUEST_URI, description = 
"The full Request URL"),
     @WritesAttribute(attribute = "http.auth.type", description = "The type of 
HTTP Authorization used"),
     @WritesAttribute(attribute = "http.principal.name", description = "The 
name of the authenticated user making the request"),
+    @WritesAttribute(attribute = "http.query.param.XXX", description = "Each 
of query parameters in the request will be added as an attribute, "
+            + "prefixed with \"http.query.param.\""),
     @WritesAttribute(attribute = HTTPUtils.HTTP_SSL_CERT, description = "The 
Distinguished Name of the requestor. This value will not be populated "
             + "unless the Processor is configured to use an SSLContext 
Service"),
     @WritesAttribute(attribute = "http.issuer.dn", description = "The 
Distinguished Name of the entity that issued the Subject's certificate. "
@@ -730,13 +732,6 @@ public class HandleHttpRequest extends AbstractProcessor {
           putAttribute(attributes, "http.server.name", 
request.getServerName());
           putAttribute(attributes, HTTPUtils.HTTP_PORT, 
request.getServerPort());
 
-          final Enumeration<String> paramEnumeration = 
request.getParameterNames();
-          while (paramEnumeration.hasMoreElements()) {
-              final String paramName = paramEnumeration.nextElement();
-              final String value = request.getParameter(paramName);
-              attributes.put("http.param." + paramName, value);
-          }
-
           final Cookie[] cookies = request.getCookies();
           if (cookies != null) {
               for (final Cookie cookie : cookies) {