Skip to content

Provide a custom CA Bundle to enable handling of self-signed SSL certs, or disable SSL verification entirely#13

Merged
simosho merged 1 commit intowaylayio:waylay_mainfrom
Really-Bad-Apps:waylay_main
Feb 27, 2024
Merged

Provide a custom CA Bundle to enable handling of self-signed SSL certs, or disable SSL verification entirely#13
simosho merged 1 commit intowaylayio:waylay_mainfrom
Really-Bad-Apps:waylay_main

Conversation

@jmazzahacks
Copy link
Copy Markdown

This seems to be the most active fork of the original (abandoned) project.

Our loki server is fronted by an SSL enabled nginx proxy, with a self managed SSL certificate. We needed to be to provide our own CA bundle to the requests library that could verify the connection without raising an SSL exception, so this PR accomplishes that with minimal impact.

The requests library accepts a verify parameter than can either be a bool (to enable / disable verification) or a string that is treated as a file path to a custom CA bundle to use for verification.

We simple extended this to the LokiHandler.

See the request lib for more details.

https://docs.python-requests.org/en/latest/user/advanced/#ssl-cert-verification

…SSL certs, or disable SSL verifiation entirely
@jmazzahacks jmazzahacks requested a review from a team as a code owner February 23, 2024 22:51
Copy link
Copy Markdown

@simosho simosho left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, thank you!

@simosho simosho merged commit 6942818 into waylayio:waylay_main Feb 27, 2024
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 this pull request may close these issues.

2 participants