Reuben Thomas
Reuben Thomas
@SuperSandro2000 apologies, I should not have pushed the previous rebase, that was just for testing purposes. I have rebased on master again, so the PR now contains just the Mythic...
Ping! I believe this is ready to go.
> Can you please fix the conflict? Sure, done!
I'm not sure why this was closed. I rebased as requested, as far as I was concerned the PR was ready. Happy to update it further if required.
@SuperSandro2000 thanks for the notification; I've rebased the PR on current `develop`. …And I seem to have messed up, so investigating.
I don't think adding an option to force the command-line method is really an improvement: it still means that there are three code paths to maintain for the same functionality....
I ran with `-debug` (though this doesn't seem to give extra information in this case), and got: ``` DEBUG: proxy = DEBUG: protocol = https DEBUG: server = ipv4.api.mythic-beasts.com DEBUG:...
> I agree... but FYI, the only way I got the command line curl accepted into the code was to also implement the module version. I really needed the command...
> You're right, this doesn't give us many clues. Is this the very first network request or has it successfully transmitted/received earlier requests? This is the first. > When I...
> All the history is in #233, you'll see it was quite a journey to get curl support added, Personally I would remove the WWW::Curl::Easy code, but it is there...