bbende commented on a change in pull request #258: NIFIREG-358 Refactoring
proxy authorization to be part of Authorizables
URL: https://github.com/apache/nifi-registry/pull/258#discussion_r378295031
##########
File path:
nifi-registry-core/nifi-registry-framework/src/main/java/org/apache/nifi/registry/security/authorization/resource/Authorizable.java
##########
@@ -211,18 +205,10 @@ default void authorize(Authorizer authorizer,
RequestAction action, NiFiUser use
userContext = null;
}
- final List<String> proxyChain = new ArrayList<>();
- NiFiUser proxyUser = user.getChain();
- while (proxyUser != null) {
- proxyChain.add(proxyUser.getIdentity());
- proxyUser = proxyUser.getChain();
- }
Review comment:
Do we consider this a breaking change?
In 0.5.0 we added the proxyIdentities field and build methods to
AuthorizationRequest. I left those there, but stopped populating them since we
don't want to use them anymore. So it won't break compilation if some was using
those, but they won't have values anymore. I can put this back if we want to
leave it.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
