dexter has submitted this change. ( https://gerrit.osmocom.org/c/onomondo-eim/+/43401?usp=email )
Change subject: cosmetic/esipa_rest_utils: re-order IpaEuiccDataJson list ......................................................................
cosmetic/esipa_rest_utils: re-order IpaEuiccDataJson list
The order of the IEs in that list are not in the same order as in the SGP.32 ASN.1 spec.
Change-Id: I6f962ce90d512369f8df37928546ec4b437d0b48 --- M src/esipa_rest_utils.erl 1 file changed, 6 insertions(+), 6 deletions(-)
Approvals: dexter: Looks good to me, approved laforge: Looks good to me, but someone else must approve Jenkins Builder: Verified
diff --git a/src/esipa_rest_utils.erl b/src/esipa_rest_utils.erl index c415692..419bc0a 100644 --- a/src/esipa_rest_utils.erl +++ b/src/esipa_rest_utils.erl @@ -531,6 +531,12 @@ case IpaEuiccDataResponse of {ipaEuiccData, IpaEuiccData} -> IpaEuiccDataJson = [ + memberOrNilAsnHex( + notificationsList, + IpaEuiccData, + 'SGP32Definitions', + 'PendingNotificationList' + ), memberOrNil(defaultSmdpAddress, IpaEuiccData), memberOrNilAsnHex( euiccInfo1, @@ -569,12 +575,6 @@ IpaEuiccData, 'RSPDefinitions', 'DeviceInfo' - ), - memberOrNilAsnHex( - notificationsList, - IpaEuiccData, - 'SGP32Definitions', - 'PendingNotificationList' ) ], IpaEuiccDataJsonFiltered =