oauth2-proxy / manifests

For hosting manifests to allow for the deployment of OAuth2-Proxy/OAuth2-Proxy
Apache License 2.0
172 stars 155 forks source link

sessionStorage.redis.sentinel.connectionUrls creates array, but expects string #267

Open Schueni1 opened 13 hours ago

Schueni1 commented 13 hours ago

connectionUrls is of wrong type. Should be a list, but expects string.

tuunit commented 12 hours ago

Thank you for contributing to the project. Please try to provide more context and a good structure when creating open source issues.

For example:

## Description
The `connectionUrls` attribute of `sessionStorage.redis.cluster` and `sessionStorage.redis.sentinel` seem to be of the wrong type.

## Expected

```yaml
sessionSotrage:
  redis:
    cluster:
      # List of Redis cluster connection URLs comma separated (e.g. `"redis://127.0.0.1:8000, redis://127.0.0.1:8001"`)
      connectionsUrls: ""
\```
tuunit commented 11 hours ago

Fixed by #268