Attention is currently required from: pespin.
laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-gsm-manuals/+/35898?usp=email )
Change subject: gsup.adoc: Add links to section of each message type in table
......................................................................
Patch Set 1: Code-Review+1
--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-manuals/+/35898?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-gsm-manuals
Gerrit-Branch: master
Gerrit-Change-Id: Idbc8a8f2e41f90e545a420517bd030399fb229ac
Gerrit-Change-Number: 35898
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 07 Feb 2024 19:33:06 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Attention is currently required from: pespin.
laforge has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-gsm-manuals/+/35897?usp=email )
Change subject: gsup.adoc: Fix Authentication Info message naming in table
......................................................................
Patch Set 1: Code-Review+1
(1 comment)
Patchset:
PS1:
I'm not entirely sure it's a fix. AFAICT even some 3GPP specs call it SendAuthInfo and not SendAuthenticationInfo. But well, we don't really need to argue about that.
--
To view, visit https://gerrit.osmocom.org/c/osmo-gsm-manuals/+/35897?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-gsm-manuals
Gerrit-Branch: master
Gerrit-Change-Id: I6d5c95334c961f1f0aa083107713c31d9894c7aa
Gerrit-Change-Number: 35897
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-Comment-Date: Wed, 07 Feb 2024 19:00:13 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Attention is currently required from: laforge, lynxis lazus.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/erlang/osmo-epdg/+/35901?usp=email )
Change subject: Make PDP-Type and APN from GSUP AuthInfoReq available through all layers
......................................................................
Patch Set 1:
(1 comment)
File src/aaa_diameter_swx.erl:
https://gerrit.osmocom.org/c/erlang/osmo-epdg/+/35901/comment/a638af84_7f39…
PS1, Line 192: %16#21 ->
idk why, if it go through this path, I get an encode error:
18:13:36.308 [debug] Swx MAR: IPv4Opt=<<"<FF><FF><FF><FE>">> IPv6Opt=[]
*DBG* aaa_ue_fsm_262420000000001 send {error,encode} to <0.174.0>
*DBG* aaa_ue_fsm_262420000000001 consume call {swm_auth_req,33,"internet"} from <0.174.0> in state state_new
18:13:36.329 [error] Error: encode
Maybe I'm encoding wrong an OctetString? Quick check in https://www2.erlang.org/documentation/doc-10.5/lib/diameter-2.2.1/doc/html/… says:
OctetString() = string() | binary()
Maybe I need to encode it as a 4 int tuple, as explained in "Derived AVP Data Formats" section just below the line I shared above.
--
To view, visit https://gerrit.osmocom.org/c/erlang/osmo-epdg/+/35901?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: erlang/osmo-epdg
Gerrit-Branch: master
Gerrit-Change-Id: I3f29b00f57f433f8623b9f0a5048c83985e5cced
Gerrit-Change-Number: 35901
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-Attention: laforge <laforge(a)osmocom.org>
Gerrit-Attention: lynxis lazus <lynxis(a)fe80.eu>
Gerrit-Comment-Date: Wed, 07 Feb 2024 18:26:15 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment