python-zeep icon indicating copy to clipboard operation
python-zeep copied to clipboard

Incompatibility with httpx 0.28.0

Open decaz opened this issue 1 year ago • 8 comments

Since version 0.28.0 of httpx The deprecated proxies argument has now been removed.:

https://github.com/mvantellingen/python-zeep/blob/41a5115774b1807735dd1ddc6e08a36ca82ce094/src/zeep/transports.py#L186-L195

decaz avatar Nov 29 '24 13:11 decaz

If anyone is experiencing this problem, simply fix your version of httpx, using an exact version pre-0.28 or the qualifier <0.28 until the attached PR is merged.

For example: httpx>=0.26,<0.28 or httpx==0.27.2

TB99930 avatar Dec 11 '24 10:12 TB99930

If anyone is experiencing this problem, simply fix your version of httpx, using an exact version pre-0.28 or the qualifier <0.28 until the attached PR is merged.

For example: httpx>=0.26,<0.28 or httpx==0.27.2

I think this is a bad solution, as other libraries may require newer version or have another constraint. Perhaps we can just initialize clients outside and pass to the Transport's constructor:

wsdl_client = httpx.Client(
    verify=sslcontext,
    timeout=HTTP_TIMEOUT,
)
httpx_client = httpx.AsyncClient(verify=sslcontext,
                                 timeout=HTTP_TIMEOUT)
transport = AsyncTransport(client=httpx_client, wsdl_client=wsdl_client)
zclient = ZeepClient(
    wsdl_file, transport=transport,
    service_name=bind_soap_service, port_name=bind_soap_port
)

ba1dr avatar Dec 11 '24 10:12 ba1dr

When will the update be?

kostyaten avatar Jan 12 '25 22:01 kostyaten

Hi @ba1dr

Could you please at least tell us whether there is any plan to fix this incompatibility issue or not?

Thanks Jose M. Prieto

PrieJos avatar Jan 23 '25 20:01 PrieJos

Hi @ba1dr

Could you please at least tell us whether there is any plan to fix this incompatibility issue or not?

Thanks Jose M. Prieto

I literally have no idea, as I am just the same user of this library as you. Looking at the past releases timeline I can only conclude that the author is busy and this library is semi-abandoned.

Perhaps eventually this will be fixed, but at the moment the best we can do is either override this issue with one of the examples above or make a fork and fix it for yourself.

ba1dr avatar Jan 29 '25 14:01 ba1dr

Hi @ba1dr

Oops, sorry for that. I don’t know why I came to the conclusion you were the developer. My apologies.

Perhaps eventually this will be fixed, but at the moment the best we can do is either override this issue with one of the examples above or make a fork and fix it for yourself.

Fully agree with you. Let’s see.

Cheers José M Prieto

PrieJos avatar Jan 29 '25 21:01 PrieJos

If anyone is experiencing this problem, simply fix your version of httpx, using an exact version pre-0.28 or the qualifier <0.28 until the attached PR is merged.

For example: httpx>=0.26,<0.28 or httpx==0.27.2

If I understand correctly, I tried both of the above suggestions in the manifest.json in the \config\custom_components\kingspan_watchman_sensit folder, but each time I restarted after the change the intergration I receive an error. Do I need to download a file from "somewhere"?

thanks in advance for any direction

K3str3l avatar Feb 06 '25 17:02 K3str3l

If anyone is experiencing this problem, simply fix your version of httpx, using an exact version pre-0.28 or the qualifier <0.28 until the attached PR is merged.

For example: httpx>=0.26,<0.28 or httpx==0.27.2

That doesn’t work for Home Assistant for example which requires latest httpx. The code from @ba1dr works well and it’s what I am doing in my HA integration.

masaccio avatar Feb 07 '25 22:02 masaccio

Are there any news on this?

alexrohozneanu avatar Aug 19 '25 06:08 alexrohozneanu