novaksolutions / infusionsoft-php-sdk

An OO Infusionsoft PHP SDK
Other
69 stars 53 forks source link

curl 7.69 #166

Closed jchimene closed 3 years ago

jchimene commented 4 years ago

Has the XMLRPC.INC been tested against curl 7.69? I think that version might be a breaking change. I have 7.64 via Debian Stable & it's working.

This seemed to fix the issue. However, this seems to point to a corrupted libcurl

OTOH, this bolsters the keepalive adjustment.

joeynovak commented 3 years ago

I couldn't find any details on why it won't work on 7.69, can you send me details on what may be a breaking change?

Joey

On Tue, Oct 13, 2020 at 10:38 AM jchimene notifications@github.com wrote:

Has the XMLRPC.INC been tested against curl 7.69? I think that version might be a breaking change. I have 7.64 via Debian Stable & it's working.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/novaksolutions/infusionsoft-php-sdk/issues/166, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAFLPKZUFO3SNEYJXY56GP3SKSF7VANCNFSM4SPKUVRA .

-- C) 503-719-2169

Like Novak Solutions on Facebook http://www.facebook.com/plugins/like.php?href=http://www.facebook.com/pages/Novak-Solutions-LLC/282048461853879

"Opportunity is missed by most people because it is dressed in overalls and looks like work." --Thomas Edison

jchimene commented 3 years ago

I can't find any details either, other than the above links.

It may be that the original poster's version of libcurl was corrupt. I can't say it wasn't.

It may be that a prophylactic change to Infusionsoft/xmlrpc.inc is indicated.

I haven't made the change on any other systems.

joeynovak commented 3 years ago

Ah, ok. I'm going to close this then and if you see the issue pop up yourself lmk.