data-dot-all / dataall

A modern data marketplace that makes collaboration among diverse users (like business, analysts and engineers) easier, increasing efficiency and agility in data projects on AWS.
https://data-dot-all.github.io/dataall/
Apache License 2.0
236 stars 82 forks source link

tenant-permission tests #1694

Closed dlpzx closed 1 week ago

dlpzx commented 2 weeks ago

Feature or Bugfix

Detail

Add unit tests that verify that MANAGE_X permissions are applied to all Mutations except for an OPT_OUT list of Mutations and to a subset of OPT_IN queries.

The OPT_OUT mutations are either:

The OPT_IN queries are operations that retrieve credentials or redirect URLs that allow the user to effectively create/update data.all objects.

Relates

Security

Please answer the questions below briefly where applicable, or write N/A. Based on OWASP 10.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

dlpzx commented 2 weeks ago

@petrkalos I have intentionally left the commented mutations and opened the PR to review. I think that the commented mutations will require some changes and I don't want to make the PR too big. If you think we should tackle all at once I am happy to keep working on this PR