esecules commented on code in PR #10079:
URL: https://github.com/apache/nifi/pull/10079#discussion_r2201304513
##########
nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/StandardNiFiServiceFacade.java:
##########
@@ -6846,18 +6846,23 @@ public FlowAnalysisResultEntity
getFlowAnalysisResult(String processGroupId) {
}
public Stream<RuleViolation> getRuleViolationStream(String processGroupId)
{
- ProcessGroup processGroup =
processGroupDAO.getProcessGroup(processGroupId);
Review Comment:
FYI the endpoint requires /flow access only and doesn't check anything else.
Let's not change it in this PR but it's worth looking into since validation
errors could expose something from a process group or processor config that
wouldn't otherwise be visible.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
