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

x509 Certificate issue #45

Closed
gipfeli opened this issue Jun 1, 2015 · 13 comments
Closed

x509 Certificate issue #45

gipfeli opened this issue Jun 1, 2015 · 13 comments
Labels

Comments

@gipfeli
Copy link

gipfeli commented Jun 1, 2015

2015/06/01 09:27:47 Get https://api.worldweatheronline.com/free/v2/weather.ashx?key=144cf08839d70a63db650f9750bb9&q=Zurich&format=json&num_of_days=3&tp=3&lang=de: x509: certificate signed by unknown authority (possibly because of "x509: cannot verify signature: algorithm unimplemented" while trying to verify candidate authority certificate "COMODO RSA Certification Authority")

Maybe adding 'x509 Certificate issue' will solve it?

@schachmat
Copy link
Owner

Can you still reproduce this, @gipfeli? WWO recently changed their certificate and it works for me. Which commit of wego are you using?

@ghost
Copy link

ghost commented Jun 1, 2015

@schachmat

Can you still reproduce this

[maxim@localhost ~]$ wego
2015/06/01 11:45:29 Get https://api.worldweatheronline.com/free/v2/weather.ashx?key=26b42a27c1503bd00132ce33bd129&q=Murmansk&format=json&num_of_days=3&tp=3&lang=en: x509: certificate signed by unknown authority (possibly because of "x509: cannot verify signature: algorithm unimplemented" while trying to verify candidate authority certificate "COMODO RSA Certification Authority")

@schachmat
Copy link
Owner

Two possibilities come to mind:

  • Your operating system is not up to date. The comodo CA needs SHA384 and RSA-4096 support to verify.
  • You don't have the comodo CA in your trust-store. Try to update the respective package (on debian it would be called ca-certificates)

A dirty workaround would be to change the we.go to use http instead of https, but I don't recommend that.

@thomshutt
Copy link

I've just started hitting this issue too on Ubuntu 14.10 with the latest version of ca-certificates

@schachmat
Copy link
Owner

Heyho @gipfeli, @Region51, @thomshutt,

can you check if it works, when you insert the line _ "crypto/sha512" to the imports in we.go? This blog entry might be related.

Also which version of golang-go are you running?

@imranzahid01
Copy link

On my mac (10.10.3) I have the following SHA checked out e0dce56 (10 days ago). go version spits out go version go1.1.2 darwin/amd64
The error I am receiving is:

2015/06/01 08:30:28 Get https://api.worldweatheronline.com/free/v2/weather.ashx: x509: certificate signed by unknown authority

@imranzahid01
Copy link

I synced my repo to master head, and added the line _ "crypto/sha512" as you mentioned, did a build install, and now wego runs smoothly.

@thomshutt
Copy link

Same here, all working after adding crypto/sha512

@gipfeli
Copy link
Author

gipfeli commented Jun 1, 2015

All working. We can close the ticket now.

@schachmat
Copy link
Owner

The issue is also fixed upstream, but I don't know in which version exactly. I am running golang 1.4.2, but debian jessie is still at 1.3.3. Im closing it, but if there are more reports from up-to-date distro users, I think we have to include that import statement in wego master temporarily, until the major distributions caught up on the golang upstream fix.

@schachmat
Copy link
Owner

Reopened, so people see this.

Still if anyone having this problem is running an up to date distribution, let me know.

@schachmat schachmat reopened this Jul 5, 2015
@OliverLeitner
Copy link

ubuntu 14.04 lts, will be "up to date" for a few years more...

go version go1.2.1 linux/amd64

installed from your github, now 5 hours ago (thats as upstream as it gets, right?).

plus: very latest security updated ssl/tls libraries.

@schachmat
Copy link
Owner

Ok, Ubuntu seems to be not bleeding edge enough unfortunately…
I have merged the respective pull request #49. It should work now without manual code adjustments after you updated wego, @OliverLeitner.

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

No branches or pull requests

5 participants