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

We don't show errors when unable to log into Docker Hub #360

Closed
ghost opened this issue Aug 7, 2018 · 13 comments · Fixed by #966
Closed

We don't show errors when unable to log into Docker Hub #360

ghost opened this issue Aug 7, 2018 · 13 comments · Fixed by #966

Comments

@ghost
Copy link

ghost commented Aug 7, 2018

When clicking "refresh" button or expanding the "Docker Hub" section under "Registries", I am always prompted for username and password and my Docker Hub registries never appear. Logging in doesn't provide any messages and never seems to work.

@jinglouMSFT
Copy link

What happens if you try to login from a command line or docker hub itself?

@ghost
Copy link
Author

ghost commented Aug 7, 2018

I can login to Docker Hub just fine via the command line and on hub.docker.com, but even after I log in via command line the VS Code Extension still prompts for password.

@StephenWeatherford
Copy link
Contributor

What OS? Also, if you do command palette -> Developer Tools, do you see any error messages in the output?

@ghost
Copy link
Author

ghost commented Aug 7, 2018

Windows 7 - 64-bit

No output at all when trying to log in.

@ghost
Copy link
Author

ghost commented Aug 7, 2018

I think I found out what it is. I finally see some output and it looks like our firewall is intercepting the SSL traffic and the cert isn't valid. I'll handle this within my company.

@ghost ghost closed this as completed Aug 7, 2018
@StephenWeatherford
Copy link
Contributor

Thanks! I'd like to keep this open because we shouldn't be silently failing.

@estebanreyl
Copy link

Its probably due to an unhandled promise rejection when the docker hub login failed. Promises that reject just silently die out. I'll take a look after incorporating feedback for my current PR.

@StephenWeatherford
Copy link
Contributor

It's likely this will be fixed by moving to AzureTreeDataProvider (https://github.com/Microsoft/vscode-azuretools/blob/940e63472fc9fd8d4b4d1bd24bc0e2a5ebf8a7d9/ui/src/treeDataProvider/AzureTreeDataProvider.ts#L27) - I believe it does appropriate error handling.

@ghost
Copy link
Author

ghost commented Aug 8, 2018

Do you know the URL that the service hits? I need to get it whitelisted in our firewall.

[Extension Host] RequestError: Error: unable to get local issuer certificate at new RequestError (C:\Users\kdoyle\.vscode\extensions\peterjausovec.vscode-docker-0.1.0\node_modules\request-promise-core\lib\errors.js:14:15) at Request.plumbing.callback (C:\Users\kdoyle\.vscode\extensions\peterjausovec.vscode-docker-0.1.0\node_modules\request-promise-core\lib\plumbing.js:87:29) at Request.RP$callback [as _callback] (C:\Users\kdoyle\.vscode\extensions\peterjausovec.vscode-docker-0.1.0\node_modules\request-promise-core\lib\plumbing.js:46:31) at self.callback (C:\Users\kdoyle\.vscode\extensions\peterjausovec.vscode-docker-0.1.0\node_modules\request\request.js:185:22) at emitOne (events.js:96:13) at Request.emit (events.js:191:7) at Request.onRequestError (C:\Users\kdoyle\.vscode\extensions\peterjausovec.vscode-docker-0.1.0\node_modules\request\request.js:877:8) at emitOne (events.js:101:20) at ClientRequest.emit (events.js:191:7) at TLSSocket.socketErrorListener (_http_client.js:358:9) at emitOne (events.js:96:13) at TLSSocket.emit (events.js:191:7) at emitErrorNT (net.js:1283:8) at _combinedTickCallback (internal/process/next_tick.js:80:11) at process._tickCallback (internal/process/next_tick.js:104:9)

@StephenWeatherford
Copy link
Contributor

Were you able to figure this out? Your question isn't specific enough. For docker hub, we're hitting URL's like https://hub.docker.com/v2/…

@ghost
Copy link
Author

ghost commented Aug 15, 2018

Thanks Stephen. I submitted a request to have this fixed. Should be good to go after that. I appreciate the help.

@StephenWeatherford StephenWeatherford changed the title Unable to log into Docker Hub We don't show errors when unable to log into Docker Hub Aug 15, 2018
@StephenWeatherford StephenWeatherford added this to the 0.2.0 milestone Aug 15, 2018
@StephenWeatherford StephenWeatherford modified the milestones: 0.2.0, 0.3.0 Aug 15, 2018
@ghost
Copy link
Author

ghost commented Aug 21, 2018

Just to add closure to the original issue. No longer failing after having the docker hub URL whitelisted in company firewall.

@StephenWeatherford
Copy link
Contributor

Thanks, appreciate your letting us know.

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

Successfully merging a pull request may close this issue.

4 participants