diff options
author | Jason Gerard DeRose <jderose@redhat.com> | 2009-02-09 13:49:00 -0700 |
---|---|---|
committer | Rob Crittenden <rcritten@redhat.com> | 2009-02-17 16:03:09 -0500 |
commit | d7a1e61da33ef0c0a0306137939cba301f1ff02a (patch) | |
tree | 65b3cca52d78d3ae8811e9dc9ca7e009491d672e | |
parent | b5b2e55be589bc97ba030d0ad070d787c646ddd0 (diff) | |
download | freeipa-d7a1e61da33ef0c0a0306137939cba301f1ff02a.tar.gz freeipa-d7a1e61da33ef0c0a0306137939cba301f1ff02a.tar.xz freeipa-d7a1e61da33ef0c0a0306137939cba301f1ff02a.zip |
Fixed a few problems in ra.py backend plugin
-rw-r--r-- | ipaserver/plugins/ra.py | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/ipaserver/plugins/ra.py b/ipaserver/plugins/ra.py index 26f4d6d9..3f820e0a 100644 --- a/ipaserver/plugins/ra.py +++ b/ipaserver/plugins/ra.py @@ -35,7 +35,7 @@ import array import errno import binascii from httplib import HTTPConnection -from urllib import urlencode +from urllib import urlencode, quote from socket import gethostname from ipalib import api, Backend @@ -95,7 +95,6 @@ class ra(Backend): xmlOutput='true', ) data = response.read() - conn.close() self.debug(data) if data is not None: request_status = self.__find_substring(data, 'header.status = "', '"') @@ -164,7 +163,7 @@ class ra(Backend): if request_type is None: request_type="pkcs10" if certificate_request is not None: - request = urllib.quote(certificate_request) + request = quote(certificate_request) request_info = "profileId=caRAserverCert&cert_request_type="+request_type+"&cert_request="+request+"&xmlOutput=true" returncode, stdout, stderr = self.__run_sslget(["-e", request_info, "-r", "/ca/ee/ca/profileSubmit", self.ca_host+":"+str(self.ca_ssl_port)]) self.debug("IPA-RA: returncode: %d" % returncode) @@ -309,7 +308,6 @@ class ra(Backend): ) self.debug("IPA-RA: response.status: %d response.reason: '%s'" % (response.status, response.reason)) data = response.read() - conn.close() self.info("IPA-RA: IPA certificate request submitted to CA: %s" % data) return ipa_certificate |