~mcepl/m2crypto#163: 'AttributeError: getreply' in M2Crypto/m2xmlrpclib.py

Migrated from: https://gitlab.com/m2crypto/m2crypto/-/issues/163
Created by: David Caro terseus@gmail.com
Created at: 2017-02-09T14:30:50.536Z

Problem present in M2Crypto 0.25.1

In the file M2Crypto/m2xmlrpclib.py line 69 there is a call to HTTPSConnection.getreply, a method that doesn't exists.

Seems like the code hasn't been correctly adapted in 730b0763eadbe14234b29a3c02ceb2988561935d when changing from HTTPS class to HTTPSConnection.

The problem is not present in M2Crypto 0.24.0

Status: REPORTED
Submitter: ~mcepl
Assigned to: No-one
Submitted: 10 months ago
Updated: 2 months ago
Labels: No labels applied.

~mcepl 10 months ago

On 2017-02-09T18:43:49.140Z, Matěj Cepl wrote:

Talking about httpslib:

Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "M2Crypto/httpslib.py", line 65, in __init__
    HTTPConnection.__init__(self, host, port, strict)
  File "/usr/lib64/python2.7/httplib.py", line 736, in __init__
    (self.host, self.port) = self._get_hostport(host, port)
  File "/usr/lib64/python2.7/httplib.py", line 777, in _get_hostport
    raise InvalidURL("nonnumeric port: '%s'" % host[i+1:])
httplib.InvalidURL: nonnumeric port: '//www.google.com'
>>>

We should have 443 as a default port.

(Last edited at 2017-02-09T18:46:57.566Z.)

~mcepl 10 months ago

On 2017-03-24T12:50:31.763Z, Matěj Cepl wrote:

It is necessary to add proper XMLRPC testing. For which we need at least simple XMLRPC server to have something to test against. Look at https://rzemieniecki.wordpress.com/2012/08/10/quick-solution-to-ssl-in-simplexmlrpcserver-python-2-6-and-2-7/

~mcepl 10 months ago

Changed on 2017-03-24T12:50:42.372Z by Matěj Cepl:

changed milestone to %8

~mcepl 10 months ago

Changed on 2017-09-18T20:27:54.871Z by Matěj Cepl:

changed milestone to 0.27

~mcepl 10 months ago

Changed on 2017-10-03T19:53:22.670Z by Matěj Cepl:

changed milestone to 0.29

~mcepl 10 months ago

Changed on 2018-02-23T21:10:32.391Z by Matěj Cepl:

removed milestone

~mcepl 4 months ago*

Certainly WONTFIX on large rewrites of XMLRPC testing. Given the status of XML-RPC lately, it is not worthy. The underlying issue should be fixed though.