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

Misleading message when unable to connect due to DNS lookup failure. #115

Open
axisofentropy opened this issue Jun 1, 2022 · 3 comments

Comments

@axisofentropy
Copy link
Member

When the CLI cannot connect to a Uffizzi API server, users may receive a misleading error message:

$ docker run --interactive --rm --tty --entrypoint=bash uffizzi/cli
root@881228c80c70:~# uffizzi login
Login to Uffizzi to view and manage your previews.
Server:  https://devopseu.uffizzicloud.com
Username:  adam.vollrath@uffizzi.com
Password:  
The command was interrupted

This message should probably tell users that the CLI could not connect. In this case it's a DNS lookup failure. (devopseu.uffizzicloud.com is a CNAME that points to another record that no longer exists.)

@gadkins gadkins added this to Ready for Development in Working Project Board Jun 3, 2022
@the38th the38th moved this from Ready for Development to In Development in Working Project Board Jun 6, 2022
@the38th
Copy link
Contributor

the38th commented Jun 7, 2022

looks like this was fixed in this commit

@the38th the38th moved this from In Development to In QA in Working Project Board Jun 7, 2022
@AnnaPochepina
Copy link

tested

@AnnaPochepina AnnaPochepina moved this from In QA to Code Review in Working Project Board Jun 9, 2022
@moklidia moklidia moved this from Code Review to Ready for Release in Working Project Board Jun 10, 2022
@the38th the38th moved this from Ready for Release to Released in Working Project Board Jun 15, 2022
@NealArw
Copy link

NealArw commented Jun 15, 2022

TESTED on prod:

  1. OK for case with invalid (unreached) URL
# uffizzi login
Login to Uffizzi to view and manage your previews.
Password:
A request was not sent to Uffizzi app

Other cases will be implement in #118

@NealArw NealArw moved this from Released to Verified on Production in Working Project Board Jun 15, 2022
@moklidia moklidia moved this from Verified on Production to Closed in Working Project Board Jun 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

No branches or pull requests

4 participants