Attention is currently required from: lynxis lazus. fixeria has posted comments on this change by lynxis lazus. ( https://gerrit.osmocom.org/c/erlang/osmo-epdg/+/41072?usp=email )
Change subject: diameter_3gpp_ts29_273_swx: add missing include for MAR ...................................................................... Patch Set 1: (2 comments) File src/aaa_diameter_swx_tests.erl: https://gerrit.osmocom.org/c/erlang/osmo-epdg/+/41072/comment/0a329009_1ea2b95a?usp=email : PS1, Line 51: AMar = {'MAR', : ["epdg.localdomain",";","1818836830",";","1",";","nonode@nohost"], : 1, % auth session state : undefined, undefined, undefined, % origin host, origin realm, dest realm : "262426399104394", : {'Vendor-Specific-Application-Id',10415,[16777265],[]}, : {'SIP-Auth-Data-Item',[],["EAP-AKA"],[],[],[],[],[],[],<<"">>,[],[],[],[]}, : There exists a record for this called `#'MAR'` in `include/diameter_3gpp_ts29_273_swx.hrl`: ```suggestion SessionId = ["epdg.localdomain",";","1818836830",";","1",";","nonode@nohost"], VAID = #'Vendor-Specific-Application-Id'{'Vendor-Id' = 10415, 'Auth-Application-Id' = [16777265]}, AMar = #'MAR'{'Session-Id' = SessionId, 'Auth-Session-State' = 1, 'User-Name' = "262426399104394", 'Vendor-Specific-Application-Id' = VAID, 'SIP-Auth-Data-Item' =, 'SIP-Number-Auth-Items' = 1, 'RAT-Type' = 1}, ``` This would have been a lot more readable. https://gerrit.osmocom.org/c/erlang/osmo-epdg/+/41072/comment/a6f6f5e8_4d8e93cf?usp=email : PS1, Line 90: = Encoded#diameter_packet.bin. You should be using `?assertEqual` and `?assertMatch` if you want to match something. -- To view, visit https://gerrit.osmocom.org/c/erlang/osmo-epdg/+/41072?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: erlang/osmo-epdg Gerrit-Branch: master Gerrit-Change-Id: I5e2cd3eaaa43f7396f6578531911c21941a3ea6b Gerrit-Change-Number: 41072 Gerrit-PatchSet: 1 Gerrit-Owner: lynxis lazus <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: fixeria <[email protected]> Gerrit-Reviewer: lynxis lazus <[email protected]> Gerrit-Reviewer: pespin <[email protected]> Gerrit-Attention: lynxis lazus <[email protected]> Gerrit-Comment-Date: Mon, 08 Sep 2025 17:36:56 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: No
