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

feat: yaml accept identity_files in local mode #78

Merged
merged 1 commit into from
Jun 10, 2022
Merged

Conversation

caarlos0
Copy link
Member

@caarlos0 caarlos0 commented Jun 9, 2022

The remote client does not use that field, wheter its set or not. Only the local client uses it. That said, it should be safe to allow it in the yaml config as well.

closes #60

Signed-off-by: Carlos A Becker <caarlos0@users.noreply.github.com>
@caarlos0 caarlos0 added the bug Something isn't working label Jun 9, 2022
@caarlos0 caarlos0 self-assigned this Jun 9, 2022
@caarlos0 caarlos0 requested a review from muesli June 9, 2022 18:07
@maaslalani
Copy link
Contributor

Don't have much context on this one, but I tested with and without the identity files config and both worked.

@caarlos0 caarlos0 merged commit b9d5128 into main Jun 10, 2022
@caarlos0 caarlos0 deleted the identity_files branch June 10, 2022 19:04
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

Successfully merging this pull request may close these issues.

Handshake failed error
2 participants