Skip to content

tests/multi_net: Update ssl_cert_rsa test cert. #12387

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

Merged
merged 1 commit into from
Sep 29, 2023

Conversation

Carglglz
Copy link
Contributor

@Carglglz Carglglz commented Sep 6, 2023

Update expired certificate, increase time validity period to five years and fix command arguments typos in commentaries.
Fixes #12370 ssl_cert_rsa.py test error.

@codecov
Copy link

codecov bot commented Sep 6, 2023

Codecov Report

Merging #12387 (e6e39e9) into master (3637252) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master   #12387   +/-   ##
=======================================
  Coverage   98.38%   98.38%           
=======================================
  Files         158      158           
  Lines       20933    20933           
=======================================
  Hits        20595    20595           
  Misses        338      338           

@dpgeorge dpgeorge added the tests Relates to tests/ directory in source label Sep 29, 2023
Update expired certificate, increase time validity period to five years and
fix command arguments typos in commentaries.

Signed-off-by: Carlos Gil <carlosgilglez@gmail.com>
@dpgeorge dpgeorge force-pushed the multinet-certs-update branch from e6e39e9 to 10f34b9 Compare September 29, 2023 07:00
@dpgeorge dpgeorge merged commit 10f34b9 into micropython:master Sep 29, 2023
@dpgeorge
Copy link
Member

Thank you!

@Carglglz Carglglz deleted the multinet-certs-update branch September 29, 2023 16:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tests Relates to tests/ directory in source
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants