Apache
/
nifi
mirror of https://github.com/apache/nifi.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

NIFI-10284 Corrected HTTP Request Authenticated User logging 

- Moved RequestAuthenticationFilter before HeaderWriterFilter

This closes #6250
Signed-off-by: Paul Grey <greyp@apache.org>
This commit is contained in:
exceptionfactory 2022-07-26 22:35:10 -05:00 committed by Paul Grey
parent 02e37713b3
commit 9bb22fcb11
No known key found for this signature in database
GPG Key ID: 8DDF32B9C7EE39D0
2 changed files with 7 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-jetty/src/main/java/org/apache/nifi/web/server/filter/StandardRequestFilterProvider.java
View File

@ -59,12 +59,12 @@ public class StandardRequestFilterProvider implements RequestFilterProvider {
final List<FilterHolder> filters = new ArrayList<>();
filters.add(getHeaderWriterFilter());
if (properties.isHTTPSConfigured()) {
filters.add(getFilterHolder(RequestAuthenticationFilter.class));
}
filters.add(getHeaderWriterFilter());
final int maxContentSize = getMaxContentSize(properties);
if (maxContentSize > MAX_CONTENT_SIZE_DISABLED) {
final FilterHolder contentLengthFilter = getContentLengthFilter(maxContentSize);

5
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-jetty/src/test/java/org/apache/nifi/web/server/filter/StandardRequestFilterProviderTest.java
View File

@ -31,6 +31,7 @@ import java.util.List;
import java.util.Map;
import java.util.Optional;
import static org.junit.jupiter.api.Assertions.assertEquals;
import static org.junit.jupiter.api.Assertions.assertFalse;
import static org.junit.jupiter.api.Assertions.assertNotNull;
import static org.junit.jupiter.api.Assertions.assertTrue;
@ -79,6 +80,10 @@ public class StandardRequestFilterProviderTest {
assertStandardFiltersFound(filters);
assertFilterClassFound(filters, RequestAuthenticationFilter.class);
final FilterHolder firstFilterHolder = filters.get(0);
final Class<? extends Filter> firstFilterClass = firstFilterHolder.getHeldClass();
assertEquals(RequestAuthenticationFilter.class, firstFilterClass);
}
private void assertStandardFiltersFound(final List<FilterHolder> filters) {