search

ansible / eda-server

Event Driven Ansible for AAP
Apache License 2.0
66 stars 42 forks source link

feat (AAP-29049) - before deleting credentials check if event streams attached #997

Closed hsong-rh closed 1 month ago

hsong-rh commented 1 month ago

https://issues.redhat.com/browse/AAP-29049

The UI will raise a warning as shown in the screenshot. Because we support the force flag, the deletion will succeed after checking Yes.

image

Dostonbek1 commented 1 month ago

@hsong-rh Does this account for resources that the user has no access to?

hsong-rh commented 1 month ago

@hsong-rh Does this account for resources that the user has no access to?

We use objects.filter to retrieve referenced resources, which should check permissions. So the answer should be no.

sonarcloud[bot] commented 1 month ago

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud