Attention is currently required from: Hoernchen.

Patch set 5:Verified +1Code-Review +2

3 comments:

Patchset:
- Patch Set #5:
  This looks ok to me. I have tried it out and it works fine. I would suggest to put the verification code into a separate module.
Commit Message:
- Patch Set #5, Line 7: smdpp: validate eid
  This is quite a complex patch. Maybe it would be helpful to write one or two lines on what it does and why it is needed? (I know this but maybe others who read the commit log have no clue.)
File osmo-smdpp.py:
- Patch Set #5, Line 133: print(f"Found GSMA permittedEins extension: {ext.oid}")
  Interesting, never saw this print(f"... thing before.

To view, visit change 40467. To unsubscribe, or for help writing mail filters, visit settings.