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

Fix slack ssl verification #30

Merged
merged 1 commit into from
Jan 15, 2021
Merged

Fix slack ssl verification #30

merged 1 commit into from
Jan 15, 2021

Conversation

nsano-rururu
Copy link
Collaborator

@nsano-rururu nsano-rururu changed the title Fixing slack ssl verification Fix slack ssl verification Jan 15, 2021
@jertel
Copy link
Owner

jertel commented Jan 15, 2021

According to these (unofficial) Python Requests docs, https://www.w3schools.com/python/ref_requests_post.asp, the verify parameter is indeed expecting a True argument if the intention is to verify certificates. Thanks for the submission.

@jertel jertel merged commit 17640fe into jertel:alt Jan 15, 2021
@nsano-rururu nsano-rururu deleted the fix-slack-ssl-verification branch January 19, 2021 11:34
@nsano-rururu nsano-rururu mentioned this pull request Jun 13, 2021
Closed
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 25, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
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
@nsano-rururu @jertel