Nico Haas
Nico Haas
I like the more generic authorization option (better for tests) but shouldn't we remove `WithAuthorizationStrategyJWT` then? Instead inserting the JWT option with/in `WithAuthorizationStrategy`. https://github.com/clouditor/clouditor/blob/aa233849bd5c7777d61d83fc1f2ff70ac2021bcb/service/orchestrator/orchestrator.go#L140-L151
When creating a new DB entry that already exists we get the following error: https://github.com/clouditor/clouditor/blob/aa233849bd5c7777d61d83fc1f2ff70ac2021bcb/persistence/gorm/gorm.go#L187-L193 https://github.com/clouditor/clouditor/blob/aa233849bd5c7777d61d83fc1f2ff70ac2021bcb/persistence/persistence.go#L35 Couldn't we return sth. like "DB entry already exists" resulting in a more readable...
Currently each API endpoint implementation doing some filtering, is implementing it on their on. See, e.g.,: https://github.com/clouditor/clouditor/blob/107d432293e82acc4b5c8879f1ca4cf9d1d95c00/service/orchestrator/assessment_results.go#L101-L116 Try to use a more generic function that depending on the request and...
**Is your feature request related to a problem? Please describe.** Be it new developers in the project or external partners who want to use the Clouditor via the API, the...
The idea is nice but wouldn't it make more sense to see how we can auto-generate this. I foresee a future where we change things in the metric and nobody...