Attention is currently required from: fixeria, pespin.
Jenkins Builder has posted comments on this change by lynxis lazus. ( https://gerrit.osmocom.org/c/erlang/osmo-epdg/+/41077?usp=email )
Change subject: aaa_diameter_swx_tests: use MAR record to improve readability
......................................................................
Patch Set 1:
(2 comments)
File src/aaa_diameter_swx_tests.erl:
Robot Comment from checkpatch (run ID ):
https://gerrit.osmocom.org/c/erlang/osmo-epdg/+/41077/comment/5e1c425e_0e84… :
PS1, Line 52: VAID = #'Vendor-Specific-Application-Id'{'Vendor-Id' = 10415,
'VAID' may be misspelled - perhaps 'VALID'?
Robot Comment from checkpatch (run ID ):
https://gerrit.osmocom.org/c/erlang/osmo-epdg/+/41077/comment/8f2be459_f0d3… :
PS1, Line 61: 'Vendor-Specific-Application-Id' = VAID,
'VAID' may be misspelled - perhaps 'VALID'?
--
To view, visit https://gerrit.osmocom.org/c/erlang/osmo-epdg/+/41077?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: I0e77aae792fb9bc2e7277668a4fdda21a993aaa2
Gerrit-Change-Number: 41077
Gerrit-PatchSet: 1
Gerrit-Owner: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-CC: Jenkins Builder
Gerrit-Attention: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Mon, 08 Sep 2025 20:37:06 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Attention is currently required from: osmith, pespin.
fixeria has posted comments on this change by fixeria. ( https://gerrit.osmocom.org/c/osmocom-bb/+/41068?usp=email )
Change subject: debian: fix empty osmocom-bb-{trxcon,virtphy} packages
......................................................................
Patch Set 1:
(1 comment)
Patchset:
PS1:
> why do we need to install into different DESTDIR?
This way it's a lot easier to describe package contents in the `*.install` files. See `debian/osmocom-bb-layer23.install`, for instance, where instead of listing each and every binary I can simply do `layer23/usr/local/bin/*`.
--
To view, visit https://gerrit.osmocom.org/c/osmocom-bb/+/41068?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: comment
Gerrit-Project: osmocom-bb
Gerrit-Branch: master
Gerrit-Change-Id: Ieee634f7f643a7599c735a5b77549cf34deb7fb9
Gerrit-Change-Number: 41068
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: osmith <osmith(a)sysmocom.de>
Gerrit-CC: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: osmith <osmith(a)sysmocom.de>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Mon, 08 Sep 2025 17:37:44 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: pespin <pespin(a)sysmocom.de>
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_1ea2… :
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_4d8e… :
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 <lynxis(a)fe80.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanitskiy(a)sysmocom.de>
Gerrit-Reviewer: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-Attention: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-Comment-Date: Mon, 08 Sep 2025 17:36:56 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No