Attention is currently required from: dexter. pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/34077 )
Change subject: SGSN_Tests: add RIM ROUTING ADDRESS field in request ...................................................................... Patch Set 3: (1 comment) Commit Message: https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/34077/comment/0bc066ce_4bb497ed PS1, Line 13: > I have changed the commit message now a bit, but I am still a bit confused. > […] I think that one thing is what the specs says it's mandatory or not, and another one it's a whole rant about how do we handle that 😄 Without looking into much details, my understanding is that the spec marks that as optional since some implementation may decide to have some sort of default route and really not care about the destination. Feel free to add a test reproducing both with and without address being present. In any case, without address it should be handled as 3G, which are not really supporting right now anyway afair. -- To view, visit https://gerrit.osmocom.org/c/osmo-ttcn3-hacks/+/34077 To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings Gerrit-Project: osmo-ttcn3-hacks Gerrit-Branch: master Gerrit-Change-Id: Id944c66f28d787a18c6c6f7c9dc885997d83e94c Gerrit-Change-Number: 34077 Gerrit-PatchSet: 3 Gerrit-Owner: dexter <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-CC: pespin <[email protected]> Gerrit-Attention: dexter <[email protected]> Gerrit-Comment-Date: Mon, 07 Aug 2023 13:02:07 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No Comment-In-Reply-To: pespin <[email protected]> Comment-In-Reply-To: dexter <[email protected]> Gerrit-MessageType: comment
