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

Fix: Explicitly point to certifi certificate chain. #26

Closed
wants to merge 3 commits into from

Conversation

htgoebel
Copy link
Contributor

@htgoebel htgoebel commented Jul 7, 2021

No description provided.

@edsu
Copy link
Owner

edsu commented May 22, 2024

I apologize it's taken so long to respond about this. Was this change actually needed?

@htgoebel
Copy link
Contributor Author

Was this change actually needed?

Yes, of course. Otherwise I would not have taken my time implementing it and creating a pull-request.

@edsu
Copy link
Owner

edsu commented May 23, 2024

Do you have an example RSS URL I can try to replicate the problem?

@htgoebel
Copy link
Contributor Author

Not after 3 years :-\

@edsu edsu closed this May 23, 2024
@edsu
Copy link
Owner

edsu commented May 23, 2024

I'm sorry it took so long for me to ask. I'm closing now since I've not had this problem.

@htgoebel
Copy link
Contributor Author

This is very disappointing. "Works for me" rarely is a good solution for an existing pull-request.

@edsu
Copy link
Owner

edsu commented Jun 3, 2024

If you aren't able to supply an example that helps me reproduce the problem I'm kind of stuck. The really disappointing thing that happened here is that it took me three years to respond, and I'm very sorry about that.

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.

None yet

2 participants