Add some XXXs to slaplib

parent ce761265
...@@ -400,7 +400,10 @@ class ComputerPartition(SlapRequester): ...@@ -400,7 +400,10 @@ class ComputerPartition(SlapRequester):
software_type = DEFAULT_SOFTWARE_TYPE software_type = DEFAULT_SOFTWARE_TYPE
request_dict = { request_dict = {
# XXX: This should be clearer that it is the id of the computer
# requesting the instance.
'computer_id': self._computer_id, 'computer_id': self._computer_id,
# XXX: Same here.
'computer_partition_id': self._partition_id, 'computer_partition_id': self._partition_id,
'software_release': software_release, 'software_release': software_release,
'software_type': software_type, 'software_type': software_type,
...@@ -634,6 +637,8 @@ class ConnectionHelper: ...@@ -634,6 +637,8 @@ class ConnectionHelper:
try: try:
self.connect() self.connect()
header_dict = {'Content-type': content_type} header_dict = {'Content-type': content_type}
# All None values or parameter_dict are converted to the String 'None'
# XXX: This is incoherent behavior to be fixed.
self.connection.request("POST", self.path + path, self.connection.request("POST", self.path + path,
urllib.urlencode(parameter_dict), header_dict) urllib.urlencode(parameter_dict), header_dict)
# If ssl error : must come from bad configuration # If ssl error : must come from bad configuration
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment