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

Error validating OIDC_POLICIES #59

Closed
micafer opened this issue Oct 16, 2024 · 2 comments
Closed

Error validating OIDC_POLICIES #59

micafer opened this issue Oct 16, 2024 · 2 comments

Comments

@micafer
Copy link

micafer commented Oct 16, 2024

There is an error in this function:

As v is an string this for get every single character of the string, so it does not work as expected.

@SerRichard
Copy link
Member

It should work, there is a config function lower in the page which converts the env variable to a list, for some reason, this is not running. I'll update the code to re-apply the variable cleaning to make sure v is a list. Additionally, the cleaning will now disregard empty strings, so the policies will become an empty list. This should resolve the issues.

That being said, I may have to take a look at why the variable is not being cleaned correctly, this may cause problems later down the line if the same occurs in the stac_collections_whitelist setting. I'll update again here when the MR is ready.

@SerRichard
Copy link
Member

It will be resolved with #60 . I will pull this to the openeo_argoworkflows repo and make a new release. Expect an updated version later this afternoon.

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

2 participants