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

Permissions not respected for configuration settings #59

Open
briddle opened this issue Oct 26, 2018 · 1 comment
Open

Permissions not respected for configuration settings #59

briddle opened this issue Oct 26, 2018 · 1 comment

Comments

@briddle
Copy link

briddle commented Oct 26, 2018

The configuration settings in:

  • Pay > Invoice template and...
  • Pay > Payment settings

show up in the settings menu even if the active backend user does not have permission to see them.

@briddle
Copy link
Author

briddle commented Dec 18, 2018

I also noticed that the Invoices controller does not check pay.access_invoices permission so any backend user can request backend/responsiv/pay/invoices and see/create/edit invoices.

The same is true for the other controllers, including those that store API-credentials (e.g. Stripe).

This is a rather serious potential breach of customer data for websites that utilise multiple backend users with different permissions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant