laforge has submitted this change. ( https://gerrit.osmocom.org/c/pysim/+/41973?usp=email )

(

1 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one. )Change subject: es9p_client: Fix type conversion in installation result notification ......................................................................

es9p_client: Fix type conversion in installation result notification

The asn.1 encoder expects bytes-like objects, we cannot simply pass hex-strings to it without conversion

Change-Id: I83ad047e043dc6b3462b188ce6dd0b2cc0e52e87 --- M contrib/es9p_client.py 1 file changed, 3 insertions(+), 3 deletions(-)

Approvals: Jenkins Builder: Verified laforge: Looks good to me, approved dexter: Looks good to me, but someone else must approve

diff --git a/contrib/es9p_client.py b/contrib/es9p_client.py index e9a71fc..c1455dc 100755 --- a/contrib/es9p_client.py +++ b/contrib/es9p_client.py @@ -128,12 +128,12 @@

if self.opts.operation == 'download': pird = { - 'transactionId': self.opts.transaction_id, + 'transactionId': h2b(self.opts.transaction_id), 'notificationMetadata': ntf_metadata, 'smdpOid': self.opts.smdpp_oid, 'finalResult': ('successResult', { - 'aid': self.opts.isdp_aid, - 'simaResponse': self.opts.sima_response, + 'aid': h2b(self.opts.isdp_aid), + 'simaResponse': h2b(self.opts.sima_response), }), } pird_bin = rsp.asn1.encode('ProfileInstallationResultData', pird)