8000 fix issue #37 regarding method location by karimabdelhakim · Pull Request #40 · cackharot/suds-py3 · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

fix issue #37 regarding method location #40

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

Merged
merged 1 commit into from
Oct 30, 2019

Conversation

karimabdelhakim
Copy link
Contributor

this issue happens when the method location/url is different to wsdl location/url.
suds client always takes the wsdl location/url and not the method location so when making a request it uses wrong location getting invalid response hence causing this issue.
by the way suds python2 library doesn't have line 107 therefore it works fine.

this fix this error
AttributeError: 'NoneType' object has no attribute 'promotePrefixes'
@drice
Copy link
drice commented Sep 9, 2019

Worked for me! Thanks!

@@ -729,7 +728,7 @@ def failed(self, binding, error):

def location(self):
p = Unskin(self.options)
return p.get('location', self.method.location)
return p.get('location', self.method.location.decode('utf-8'))
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is it better to leave this as .decode()?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it is the same as utf-8 is the default encoding as stated in python3 docs

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is there a benefit to duplicating that?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no

@halitcelik
Copy link

what exactly should I write in my project's requirements.txt file to be able to install this pull request instead of "suds-py3==1.3.3.0"?

@karimabdelhakim
Copy link
Contributor Author

requirements.txt git+https://github.com/karimabdelhakim/suds-py3#egg=suds-py3
to install pip install git+https://github.com/karimabdelhakim/suds-py3#egg=suds-py3
you can also check here

@halitcelik
Copy link

Thank you very much, I had changed it manually and replaced it with what you said, worked!

lsaffre added a commit to lino-framework/welfare that referenced this pull request Oct 2, 2019
@lsaffre
Copy link
lsaffre commented Oct 2, 2019

Thanks for this. I had the same issue and could fix it as explained here.

8000
@mlandry8
Copy link

merge this bb

Copy link
@theplen theplen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested and works

@cackharot cackharot merged commit b9b3bc5 into cackharot:master Oct 30, 2019
hatchjdecho added a commit to hatchjdecho/suds-py3 that referenced this pull request Nov 4, 2019
cackharot added a commit that referenced this pull request Mar 5, 2020
Resolves issue #45 fix required as result of change in #40
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.

7 participants
0