Attention is currently required from: laforge.
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-iuh/+/36309?usp=email )
Change subject: Add ranap_decode_rab_releaseditemies_fromlist()
......................................................................
Patch Set 1: Code-Review+1
(1 comment)
File include/osmocom/ranap/ranap_common.h:
https://gerrit.osmocom.org/c/osmo-iuh/+/36309/comment/c303ffda_5cb0a453
PS1, Line 666: int ranap_decode_rab_releaseditemies_fromlist(RANAP_RAB_ReleasedItemIEs_t *raB_ReleasedItemIEs, ANY_t *any_p);
probably wanna follow the one field per line of the above, so that it looks similar.
--
To view, visit https://gerrit.osmocom.org/c/osmo-iuh/+/36309?usp=email
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings
Gerrit-Project: osmo-iuh
Gerrit-Branch: master
Gerrit-Change-Id: I328819c650fc6cefe735093a846277b4f03e6b29
Gerrit-Change-Number: 36309
Gerrit-PatchSet: 1
Gerrit-Owner: laforge <laforge(a)osmocom.org>
Gerrit-Reviewer: pespin <pespin(a)sysmocom.de>
Gerrit-CC: Jenkins Builder
Gerrit-Attention: laforge <laforge(a)osmocom.org>
Gerrit-Comment-Date: Fri, 15 Mar 2024 18:17:42 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment
Hello Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/erlang/osmo-epdg/+/36305?usp=email
to look at the new patch set (#4).
The following approvals got outdated and were removed:
Verified+1 by Jenkins Builder
Change subject: Propagate SWx PPR as SWm Authorization Update towards ePDG
......................................................................
Propagate SWx PPR as SWm Authorization Update towards ePDG
With this commit ePDG is still not invoking the authorization procedure
specified in 3GPP TS 29.273 7.1.2.2, but only ACKing the RAR message
from 7.1.2.5.
Related: OS#6404, OS#6400.
Change-Id: I4fe309ac54c61f134dbd471486c7d1a70731c9bb
---
M src/aaa_diameter_swm.erl
M src/aaa_diameter_swx_cb.erl
M src/aaa_ue_fsm.erl
M src/epdg_diameter_swm.erl
M src/epdg_ue_fsm.erl
5 files changed, 147 insertions(+), 12 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/erlang/osmo-epdg refs/changes/05/36305/4
--
To view, visit https://gerrit.osmocom.org/c/erlang/osmo-epdg/+/36305?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: I4fe309ac54c61f134dbd471486c7d1a70731c9bb
Gerrit-Change-Number: 36305
Gerrit-PatchSet: 4
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-MessageType: newpatchset
pespin has posted comments on this change. ( https://gerrit.osmocom.org/c/erlang/osmo-epdg/+/36305?usp=email )
Change subject: Propagate SWx PPR as SWm Authorization Update towards ePDG
......................................................................
Patch Set 3:
This change is ready for review.
--
To view, visit https://gerrit.osmocom.org/c/erlang/osmo-epdg/+/36305?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: I4fe309ac54c61f134dbd471486c7d1a70731c9bb
Gerrit-Change-Number: 36305
Gerrit-PatchSet: 3
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Comment-Date: Fri, 15 Mar 2024 16:28:52 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
Attention is currently required from: pespin.
Hello Jenkins Builder,
I'd like you to reexamine a change. Please visit
https://gerrit.osmocom.org/c/erlang/osmo-epdg/+/36305?usp=email
to look at the new patch set (#2).
The following approvals got outdated and were removed:
Verified-1 by Jenkins Builder
Change subject: Propagate SWx PPR as SWm Authorization Update towards ePDG
......................................................................
Propagate SWx PPR as SWm Authorization Update towards ePDG
With this commit ePDG is still not invoking the authorization procedure
specified in 3GPP TS 29.273 7.1.2.2, but only ACKing the message.
Related: OS#6404, OS#6400.
Change-Id: I4fe309ac54c61f134dbd471486c7d1a70731c9bb
---
M src/aaa_diameter_swm.erl
M src/aaa_diameter_swx_cb.erl
M src/aaa_ue_fsm.erl
M src/epdg_diameter_swm.erl
M src/epdg_ue_fsm.erl
5 files changed, 131 insertions(+), 13 deletions(-)
git pull ssh://gerrit.osmocom.org:29418/erlang/osmo-epdg refs/changes/05/36305/2
--
To view, visit https://gerrit.osmocom.org/c/erlang/osmo-epdg/+/36305?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: I4fe309ac54c61f134dbd471486c7d1a70731c9bb
Gerrit-Change-Number: 36305
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Attention: pespin <pespin(a)sysmocom.de>
Gerrit-MessageType: newpatchset
Jenkins Builder has posted comments on this change. ( https://gerrit.osmocom.org/c/erlang/osmo-epdg/+/36305?usp=email )
Change subject: Propagate SWx PPR as SWm Authorization Update towards ePDG
......................................................................
Patch Set 1:
(1 comment)
File src/aaa_ue_fsm.erl:
Robot Comment from checkpatch (run ID jenkins-gerrit-lint-15152):
https://gerrit.osmocom.org/c/erlang/osmo-epdg/+/36305/comment/e7dc5a41_e658…
PS1, Line 346: %% SWx PPA was already answered immediatelly when PPR was received, nothing to do here.
'immediatelly' may be misspelled - perhaps 'immediately'?
--
To view, visit https://gerrit.osmocom.org/c/erlang/osmo-epdg/+/36305?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: I4fe309ac54c61f134dbd471486c7d1a70731c9bb
Gerrit-Change-Number: 36305
Gerrit-PatchSet: 1
Gerrit-Owner: pespin <pespin(a)sysmocom.de>
Gerrit-CC: Jenkins Builder
Gerrit-Comment-Date: Fri, 15 Mar 2024 15:28:08 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Gerrit-MessageType: comment