The attached patch adds a permission check on the "Configure" link in the tray.

CommentFileSizeAuthor
#3 after patch.png13.37 KBpratik_kamble
#3 Without patch.png15.41 KBpratik_kamble
environment_indicator-permission.patch1.11 KBprudloff
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

prudloff created an issue. See original summary.

pratik_kamble’s picture

pratik_kamble
Location Pune, India
CreditAttribution: pratik_kamble at QED42 commented
Assigned: prudloff » pratik_kamble
pratik_kamble’s picture

pratik_kamble
Location Pune, India
CreditAttribution: pratik_kamble at QED42 commented
FileSize
Without patch.png15.41 KB
after patch.png13.37 KB

@prudloff, thanks for the patch. I have reviewed the patch. It LGTM.

Without the patch, user is able to see the Configure link in the tray for the user who doesn't have permission to access the page.
After applying the patch user is not able to see the configure link in the tray.
PFA for reference.

pratik_kamble’s picture

pratik_kamble
Location Pune, India
CreditAttribution: pratik_kamble at QED42 commented
Assigned: pratik_kamble » Unassigned
Status: Needs review » Reviewed & tested by the community

pratik_kamble’s picture

pratik_kamble
Location Pune, India
CreditAttribution: pratik_kamble at QED42 commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.