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

Hacked log handler for 'suds.transport' #33

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

jedie
Copy link
Contributor

@jedie jedie commented Feb 20, 2019

I haved made a simple log handler for 'suds.transport' that will print the binary output a little bit better to the console.

The normal output:

grafik

With this hacked handler:

grafik

Only tested for Python 3... Maybe Python 2 needs some extra handling...

@jedie
Copy link
Contributor Author

jedie commented Feb 20, 2019

What's about to move this out of sandbox into django-oscar-docdata/oscar_docdata ?

@maerteijn
Copy link
Member

I'd say that this is suds (and your way of debugging) specific and not really something that belongs inoscar_docdata

@jedie
Copy link
Contributor Author

jedie commented Feb 21, 2019

Yes, you are right: The best place for this is the suds project.

But currently we use https://bitbucket.org/jurko/suds and we should switched to https://github.com/suds-community/suds #15

So i will create a pull request in https://github.com/suds-community/suds see also: suds-community/suds#17

But maybe we should leave this in oscar-docdata until the changeover to suds-community is complete?

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

Successfully merging this pull request may close these issues.

2 participants