dexter has submitted this change. ( https://gerrit.osmocom.org/c/onomondo-eim/+/42876?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: esipa_asn1_handler: fix transactionId handling ......................................................................

esipa_asn1_handler: fix transactionId handling

The transactionId member is now called eimTransactionId. It is also an optional member.

Change-Id: I1340e0c7aed091e8e5a5089ebd6e524512408cf7 Related: SYS#8100 --- M src/esipa_asn1_handler_utils.erl 1 file changed, 4 insertions(+), 4 deletions(-)

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

diff --git a/src/esipa_asn1_handler_utils.erl b/src/esipa_asn1_handler_utils.erl index 5bf47da..c297638 100644 --- a/src/esipa_asn1_handler_utils.erl +++ b/src/esipa_asn1_handler_utils.erl @@ -14,21 +14,21 @@ EuiccPackageResultDataSigned = maps:get( euiccPackageResultDataSigned, EuiccPackageResultSigned ), - maps:get(transactionId, EuiccPackageResultDataSigned); + maps:get(eimTransactionId, EuiccPackageResultDataSigned, none); {euiccPackageErrorSigned, EuiccPackageErrorSigned} -> EuiccPackageErrorDataSigned = maps:get( euiccPackageErrorDataSigned, EuiccPackageErrorSigned ), - maps:get(transactionId, EuiccPackageErrorDataSigned); + maps:get(eimTransactionId, EuiccPackageErrorDataSigned, none); _ -> none end.

process_euiccPackageResult(Pid, EuiccPackageResult, EsipaReq, TransactionId) -> WorkBind = fun(Map) -> - case maps:is_key(transactionId, Map) of + case maps:is_key(eimTransactionId, Map) of true -> - mnesia_db:work_bind(Pid, maps:get(transactionId, Map)); + mnesia_db:work_bind(Pid, maps:get(eimTransactionId, Map)); _ -> ok end