Attention is currently required from: jolly.
dexter has posted comments on this change by jolly. ( https://gerrit.osmocom.org/c/onomondo-ipa/+/43035?usp=email )
Change subject: V1.2: Add eimTransactionId to IpaEuiccDataRequest and IpaEuiccDataResponse ......................................................................
Patch Set 1:
(2 comments)
File src/ipa/libipa/proc_euicc_data_req.c:
https://gerrit.osmocom.org/c/onomondo-ipa/+/43035/comment/6c7008df_35acebcc?... : PS1, Line 211: I think at this point it is too late to set the eimTransactionId. You have to do this before the ipa_esipa_prvde_eim_pkg_rslt() call. Otherwise there will be no effect. (as it seems the if/else statement below was intended for producing log messages only)
https://gerrit.osmocom.org/c/onomondo-ipa/+/43035/comment/bad2f18e_a0175937?... : PS1, Line 216: IPA_LOGP(SIPA, LINFO, "IPA get EUICC data succeeded!\n"); When looking closer I can see that this code is a bit misleading. At the moment we do not generate an ipaEuiccDataResponseError. If one of those ES10b function calls above fails, there will just be no result for this tag. Maybe this is something we should address. (create a ticket?)