-
Notifications
You must be signed in to change notification settings - Fork 56
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
Display original error on failure #49
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@djc This looks like an uncontroversial fix that might be nice to clear out of the PR queue.
There was a Macos CI failure but the job logs have already been purged and I don't have permissions to re-run. Could you kick the build and see if it was something transient?
It's interesting that the I tried running it a bunch of times in rapid succession locally, but it passed every time. |
I was curious (and already looking at some CI failures today) so I tried hammering apple.com a bit from a test GHA repo. I was able to reproduce the issue with |
@ctz do you remember if there's a specific reason to include apple.com here? Do we suspect that they run a SecureTransport-based server which we'd like to try, or something? |
(Meanwhile, we probably shouldn't block this PR on the intermittent CI issues.) |
It's really just a random selection of big company websites that came to mind at the time. |
Okay, maybe we should just get rid of the |
No description provided.