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.

Sign up for GitHub

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

Jump to bottom

Ensure certifi as default tls_server_trusted_certs_cb #41

Merged
merged 3 commits into from
Sep 24, 2024
Merged

Ensure certifi as default tls_server_trusted_certs_cb #41

merged 3 commits into from
Sep 24, 2024

Conversation

ziopio
Copy link
Member

@ziopio ziopio commented Sep 24, 2024 •
edited
Loading

tls_server_trusted_certs_cb is required by grisp_cryptoauth

  • Define certifi:cacerts on boot if no option is defined by the user
  • Document this behaviour and point to grisp_cryptoauth doc for reference
  • Set empty lists in the other configurations to avoid certifi outside the prod profile
  • Restored certifi as fixed dependency

@ziopio
Fix typo
6308696
@ziopio
Ensure tls_server_trusted_certs_cb is set to certifi by default
698aa15 
dev local and test configs now need to define an empty list to disable certifi usage
@ziopio ziopio merged commit 5733297 into main Sep 24, 2024
1 check passed
@ziopio ziopio deleted the ziopio/default-ca branch September 24, 2024 09:26
@ziopio ziopio mentioned this pull request Oct 16, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@GwendalLaurent GwendalLaurent GwendalLaurent approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ziopio @GwendalLaurent