Github user patricker commented on a diff in the pull request:
https://github.com/apache/nifi/pull/2899#discussion_r203389775
--- Diff:
nifi-nar-bundles/nifi-framework-bundle/nifi-framework/nifi-web/nifi-web-ui/src/main/webapp/js/nf/canvas/nf-canvas.js
---
@@ -167,12 +167,14 @@
nfNgBridge.injector.get('breadcrumbsCtrl').resetScrollPosition();
// set page title to the name of the root processor group
- var rootBreadcrumb = breadcrumb;
- while(rootBreadcrumb.parentBreadcrumb != null) {
- rootBreadcrumb = rootBreadcrumb.parentBreadcrumb
- }
+ if (breadcrumb.permissions.canRead) {
+ var rootBreadcrumb = breadcrumb;
+ while(rootBreadcrumb.parentBreadcrumb != null) {
+ rootBreadcrumb = rootBreadcrumb.parentBreadcrumb
+ }
- document.title = rootBreadcrumb.breadcrumb.name;
+ document.title = rootBreadcrumb.breadcrumb.name;
--- End diff --
I really appreciate your deep understanding of these more corner case
security issues!
I'm thinking on it, will hope to have it updated Friday.
---
