Github user mcgilman commented on a diff in the pull request: https://github.com/apache/nifi/pull/2703#discussion_r194495873 --- Diff: nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-api/src/main/java/org/apache/nifi/web/controller/ControllerFacade.java --- @@ -1389,104 +1420,119 @@ private ProvenanceEventDTO createProvenanceEventDto(final ProvenanceEventRecord // sets the component details if it can find the component still in the flow setComponentDetails(dto); - // only include all details if not summarizing - if (!summarize) { - // convert the attributes - final Comparator<AttributeDTO> attributeComparator = new Comparator<AttributeDTO>() { - @Override - public int compare(AttributeDTO a1, AttributeDTO a2) { - return Collator.getInstance(Locale.US).compare(a1.getName(), a2.getName()); - } - }; +// try { +// AuthorizationResult result = flowController.checkConnectableAuthorization(event.getComponentId()); + AuthorizationResult result = checkConnectableAuthorization(event.getComponentId()); --- End diff -- Why not check the authorization within `setComponentDetails`? In there you already have the components to authorize and you'll know the corresponding type.
---