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
xmlrpc.client: Fault.faultCode erroneously documented to be a string, should be int #87520
Comments
I created a XMLRPC API (via Zope), which gets consumed by a C# application. C#'s XMLRPC lib expects an This leads to a The documentation for I had a look at the cpython source code, and at least the tests all use ints for eg cpython/Lib/test/test_xmlrpc.py Line 166 in 255f53b
Having a look at the XMLRPC spec at http://xmlrpc.com/spec.md it is clearly shown that Typeshed recently added type hints for xmlrpc lib, and they used string for I suggest to both update cpython's documentation and to create a PR for typeshed in order to fix the type issue. If any core dev agrees on this, I'd like to prepare the pull requests. Thanks for taking your time to look into this! |
The evidence you have here seems pretty compelling and this change seems straightforward enough. I don't see an expert listed here, but I'm happy to merge a docs PR fixing this. Probably a good idea to make a PR to typeshed in parallel, in case they have some further insight into this. |
The approved typeshed PR: |
This was merged already. I did not know I have to manually close the issue, but here we go! |
Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.
Show more details
GitHub fields:
bugs.python.org fields:
The text was updated successfully, but these errors were encountered: