Bug #6383
closedpysim shell: edit_decoded EF.ePDGId is missing the epdg address
0%
Description
I'm trying to edit the EF.ePDGId:
select ADF.USIM
select EF.ePDGId
edit_binary_decoded
# before edit, because file is "empty"
change to
{
"e_pdg_id": {
"type_of_ePDG_address": 255,
"ePDG_address": null
}
}
{
"e_pdg_id": {
"type_of_ePDG_address": 0,
"ePDG_address": "epdg.osmocom.org"
}
}
Results into 800100.
The domain is completely missing.
Updated by fixeria about 2 months ago
lynxis can you try the following?
{
"e_pdg_id": {
"type_of_ePDG_address": "FQDN", // <<---
"ePDG_address": "epdg.osmocom.org"
}
}
I think I also faced this limitation of trying to pass a value instead of an enum string.
Updated by fixeria about 2 months ago
Correction (edited): "FQDN" instead of "IPv4", of course.
Updated by fixeria about 2 months ago
With this patch (not a fix, just a proposal / RFC):
https://gerrit.osmocom.org/c/pysim/+/36140 [RFC] ePDGId: allow passing both string and int type values
you should be able to encode the FQDN either way: by passing "FQDN" or 0.
Updated by laforge about 2 months ago
- Status changed from New to Rejected
fixeria wrote in #note-3:
With this patch (not a fix, just a proposal / RFC): https://gerrit.osmocom.org/c/pysim/+/36140 you should be able to encode the FQDN either way: by passing "FQDN" or 0.
I really don't think this is the way to go. I think it's not worth the massive extra effort, both in modifying the code as well as generating test cases for it. Our promise is that whatever we generate during decode can be re-encoded. It's big enough a challenge to support that. Adding multiple different input formats just makes things a hell more complex, particularly as you can not simpyl do a decode -> reencode test, as the decode would always give you the string values.
I think the proper solution to this problem is to have good JSON examples as part of our user manual. Those could be auto-generated from test case input data that we [should] have for each file, anyway. See #6316
