Skip to content
This repository has been archived by the owner on May 7, 2024. It is now read-only.

add --silence-http-endpoint-warning flag #582

Merged
merged 4 commits into from Sep 4, 2020

Conversation

ubergesundheit
Copy link
Member

Towards giantswarm/giantswarm#12975
Requires giantswarm/gscliauth#25

This PR adds a --silence-http-endpoint-warnings flag which disables Warning: endpoint URL uses an insecure protocol messages printed when using gsctl with a http endpoint.

@ubergesundheit ubergesundheit requested a review from a team September 4, 2020 10:38
@coveralls
Copy link

coveralls commented Sep 4, 2020

Coverage Status

Coverage decreased (-0.08%) to 51.624% when pulling 14f7f57 on add-silence-http-endpoint-error-flag into 3943524 on master.

Copy link
Contributor

@axbarsan axbarsan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking good

commands/root.go Outdated Show resolved Hide resolved
commands/root.go Outdated Show resolved Hide resolved
@ubergesundheit ubergesundheit merged commit 832ff76 into master Sep 4, 2020
@ubergesundheit ubergesundheit deleted the add-silence-http-endpoint-error-flag branch September 4, 2020 11:26
@marians
Copy link
Member

marians commented Sep 8, 2020

What was the impact of this change regarding UX? Did the output of authentication related errors or log messages change in any way?

@ubergesundheit
Copy link
Member Author

All warnings and errors from gscliauth are silenced when the --silence-http-endpoint-warnings is specified.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
4 participants