Michaela Lang

Results 42 comments of Michaela Lang

added unittests for exception handling checks introduced in this PR

Codecov doesn't recognize that the unittests do cover the newly added code. I do not understand how I should fix the codecov/patch check to be satisfied. For a better understanding...

As mentioned previously ... the Unittest for entity searching requires to be updated to first `federate` (link) the searched user into the Database. Something like seen below. In other words,...

@BillDett can someone from the team verify and approve the PR please ? thanks Michi

@dmage we have a typo in the secret name `spec.template.spec.containers[0].volumeMounts[3].name: Not found: \"postgres-cert-store\"` where `postgres-cert-store` should be -postgres-certs-store` with an `s`. can we fix that without a PR or is...

Unrelated Cypress tests failing ... again :D ``` │ ✖ repository-autopruning.cy.ts 01:05 14 13 1 - - │ ├────────────────────────────────────────────────────────────────────────────────────────────────┤ │ ✔ repository-delete.cy.ts 00:04 1 1 - - - │ ├────────────────────────────────────────────────────────────────────────────────────────────────┤...

@Kamillaova I am working on a authorization "proxy" for fine grained queries. They would still be exposed but that's for all the tools (perses, grafana, ..) but restricted based upon...

The cypress test failing is not related to the change commited ... ``` │ ✖ repository-autopruning.cy.ts 01:03 14 13 1 - - │ ```

Quote: Why not modify the `/config` endpoint that exists for the same reason? This is not a super user thing. Because it should be a protected endpoint and only available...

@ibazulic but that should be `/config` not `/api/v1/superuser/config` ...