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

Mosquitto CA test file expired? #230

Closed
HerrMuellerluedenscheid opened this issue Jan 10, 2021 · 2 comments · May be fixed by #232
Closed

Mosquitto CA test file expired? #230

HerrMuellerluedenscheid opened this issue Jan 10, 2021 · 2 comments · May be fixed by #232

Comments

@HerrMuellerluedenscheid

Hey @njouanin,
I ran the nosetests locally and ran into an SSL error:

ssl.SSLCertVerificationError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: CA certificate key too weak (_ssl.c:1051)
asyncio: ERROR: SSL error in data received
protocol: <asyncio.sslproto.SSLProtocol object at 0x7ff59855fba8>
transport: <_SelectorSocketTransport closing fd=150 read=idle write=<idle, bufsize=0>>
Traceback (most recent call last):
  File "/home/marius/.pyenv/versions/3.7.1/lib/python3.7/asyncio/sslproto.py", line 526, in data_received
    ssldata, appdata = self._sslpipe.feed_ssldata(data)
  File "/home/marius/.pyenv/versions/3.7.1/lib/python3.7/asyncio/sslproto.py", line 189, in feed_ssldata
    self._sslobj.do_handshake()
  File "/home/marius/.pyenv/versions/3.7.1/lib/python3.7/ssl.py", line 763, in do_handshake
    self._sslobj.do_handshake()
ssl.SSLCertVerificationError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: CA certificate key too weak (_ssl.c:1051)
hbmqtt.client: WARNING: MQTT connection failed: SSLCertVerificationError(1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: CA certificate key too weak (_ssl.c:1051)')

Does the tests/mosquitto.org.crt maybe need to be renewed?

Best regards
Marius

@HerrMuellerluedenscheid
Copy link
Author

I can confirm that the certificate is expired. I'm going to open a MR that downloads the certificate on the fly.

@HerrMuellerluedenscheid
Copy link
Author

#232 resolves this issue. Thus, closing.

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

Successfully merging a pull request may close this issue.

1 participant