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

add OpenSSL error to rescue list #47

Merged
merged 1 commit into from
Sep 3, 2024
Merged

Conversation

bmos
Copy link
Contributor

@bmos bmos commented Apr 17, 2024

fixes #31

SSL errors will no longer cause builds to fail, leaving partial results.
Only the feed with broken SSL (or the other supported errors) will be skipped.

@geraldb
Copy link
Member

geraldb commented Apr 17, 2024

@bmos thks for the openssl fix. thks for helping out. i try to get back to pluto (dev). thks for your patience.

@bmos
Copy link
Contributor Author

bmos commented Sep 2, 2024

Hi Gerald,
I just wanted to bump this one since it has caused some bugs in projects where it's used (such as OpenSUSE Planet). The others are just improvements, but this PR should be really straightforward and impactful.
I hope you are having a good summer.

@geraldb
Copy link
Member

geraldb commented Sep 3, 2024

sorry for the wait. looks good. thanks for the reminder. i publish a new pluto-feedfetcher gem with your update in the next minutes.

update - @bmos fyi - new gem with your patch / fix now uploaded. see https://rubygems.org/gems/pluto-feedfetcher/versions/0.1.7

@geraldb geraldb merged commit 0c0f271 into feedreader:master Sep 3, 2024
@bmos bmos deleted the patch-3 branch September 3, 2024 11:20
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.

error: SSL_connect
2 participants