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

port configuration option is not configuring Dex's command #210

Open
DnPlas opened this issue Jul 22, 2024 · 1 comment
Open

port configuration option is not configuring Dex's command #210

DnPlas opened this issue Jul 22, 2024 · 1 comment
Labels
bug Something isn't working

Comments

@DnPlas
Copy link
Contributor

DnPlas commented Jul 22, 2024

Bug Description

Despite having a configuration option port for changing Dex's "listening port", this value is not used at all by Dex's command; if users change this value via the config option, Dex will not use it at all, as it is not reflected in the service command

Also, calling it "listening port" is misleading, according to the documentation, it is "The HTTP port used by Dex, default is 5556. Used to define web -> http in the Dex configuration as well as OIDC configurations". The configuration description should be more clear about it.

We should change this by making the proper refactor in the code and in the rock.

@DnPlas DnPlas added the bug Something isn't working label Jul 22, 2024
Copy link

Thank you for reporting us your feedback!

The internal ticket has been created: https://warthogs.atlassian.net/browse/KF-6034.

This message was autogenerated

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant