Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Manage navigation entry is not hidden if the user does not have ROLE_MANAGE #1843

Open
klopfdreh opened this issue Apr 11, 2022 · 6 comments
Labels
status/need-triage Team needs to triage and take a first look

Comments

@klopfdreh
Copy link
Contributor

Hey,

I was expecting the menu entry to be hidden when the user does not have the role "ROLE_MANAGE" in the spring cloud data flow server ui:

image

My question is how I can make this entry not visible with "Customizing Authorization" mentioned here: https://docs.spring.io/spring-cloud-dataflow/docs/current/reference/htmlsingle/#configuration-security-customizing-authorization

kind regards and thanks in advance.

@github-actions github-actions bot added the status/need-triage Team needs to triage and take a first look label Apr 11, 2022
@klopfdreh
Copy link
Contributor Author

@oodamien
Copy link
Member

oodamien commented Apr 20, 2022

Hi @klopfdreh,
ROLE_MANAGE is not meant to be used on the UI part (see the definition you pointed).
We will have in the following days a discussion with the team to see how we can answer to this request.

@klopfdreh
Copy link
Contributor Author

Hey @oodamien,
yes I saw it - online for the spring boot admin actuator endpoints. Thanks for the information, I wait till your answer. 👍

@markpollack
Copy link
Contributor

Thanks for the input, we are going to review this functionality and look into defining new roles that will make these menu options available that are separate from the current use of the role_manage for actuator endpoints.

@klopfdreh
Copy link
Contributor Author

Sounds great! Looking forward for that implementation. You can add me as reviewer. 👍

@klopfdreh
Copy link
Contributor Author

Hey @markpollack / @oodamien - are there any updates when this feature is going to be implemented?

Currently I am patching the UI manually so that the Manage menu entry is not shown when you don't have the ROLE_MANAGE.

Unfortunately, this is not very secure as this entry is only hidden but you can still access it when you know the path.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status/need-triage Team needs to triage and take a first look
Projects
None yet
Development

No branches or pull requests

3 participants