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

suds wordt (te) oud #160

Closed
Wim-De-Clercq opened this issue Sep 6, 2021 · 3 comments
Closed

suds wordt (te) oud #160

Wim-De-Clercq opened this issue Sep 6, 2021 · 3 comments

Comments

@Wim-De-Clercq
Copy link
Contributor

Wim-De-Clercq commented Sep 6, 2021

We gebruiken suds-jurko als soap client. Is sinds 2014 niet geupdate. Zit vol python 2 code en hangt af van een library binnen python 2to3 om bij het installeren van de library in een python 3 omgeving automatisch de code python 3 compatibel te maken.

Dit was een gesupporteerde feature binnen setuptools. Maar bij de nieuwe setsuptools is die (eerst deprecated geweest en later is de) support verwijderd.
pypa/setuptools#2086

Gevolg is dat wij bij onze projecten die crabpy/suds gebruiken ook niet de nieuwste setuptools kunnen gebruiken.

De error die je mogelijk te zien krijgt is

    from crabpy_pyramid import ICapakey
../../../virtualenv/python3.6.7/lib/python3.6/site-packages/crabpy_pyramid/__init__.py:7: in <module>
    from crabpy.client import crab_factory
../../../virtualenv/python3.6.7/lib/python3.6/site-packages/crabpy/client.py:13: in <module>
    from suds.client import Client
../../../virtualenv/python3.6.7/lib/python3.6/site-packages/suds/__init__.py:28: in <module>
    from version import __build__, __version__
E   ModuleNotFoundError: No module named 'version'

Bij nieuwe virtualenvs vanaf 5 september!
image

@OnroerendErfgoed/applicatiebeheerders Ik tag jullie omdat jullie dit mogelijks zullen voorhebben bij deploys, als de virtualenv nieuw is.
Workaround is pip install setuptools==57.5.0 in de virutalenv te doen.

@cedrikv
Copy link
Contributor

cedrikv commented Sep 6, 2021

ideaal voor als ee nsprint zodner tickets dreigt te vallen. Kan je er een (ruwe) estimate op plakken?

@Wim-De-Clercq
Copy link
Contributor Author

goh, kweet het niet goed. Deze 2 files ken ik eigenlijk niet, en ik zie niet welk project die gebruikt (Ik hoop eigenlijk dat we die gewoon kunnen deleten)
https://github.com/OnroerendErfgoed/crabpy/blob/master/crabpy/wsa.py
https://github.com/OnroerendErfgoed/crabpy/blob/master/crabpy/wsse.py

Als we die 2 even negeren is de change hier eigenlijk heel klein op suds door zeep te veranderen. Gelijk 2-3 puntjes maar.
De grotere changes zouden eerder bij crabpy_pyramid zitten, die de client gebruikt. Maar op het eerste zicht lijken zelfs daar niet echt changes nodig te zijn ... als ik eens door de readme van zeep kijk en vergelijk met onze code.

Omdat ik die 2 files niet ken, zou ik er dan een 3 op zetten.

@Wim-De-Clercq
Copy link
Contributor Author

Wim-De-Clercq commented Sep 7, 2021

Zoals emrys heeft gemeld bestaat er een "suds-py3" library. We zullen die gebruiken in de plaats voor nu. Is eenvoudige upgrade.

Extra note: Bestaande virtualenvironenments waar suds-jurko bestaat gaan die niet automatisch verwijderen. Ik raad dus aan om ofwel een nieuwe te maken of manueel pip uninstall suds-jurko te doen.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants