laforge has submitted this change. ( https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/40903?usp=email )
(
1 is the latest approved patch-set. No files were changed between the latest approved patch-set and the submitted one. )Change subject: sctp_proxy: fix non-existent fields in fetch_info/0 ......................................................................
sctp_proxy: fix non-existent fields in fetch_info/0
The mme_addr and mme_port have been moved into the conn_cfg sub-map.
Change-Id: I386a0268d5e465cbd5ad7ee02ed768425e6ec17f Fixes: 1221b6d ("sctp_{client,server}: accept arguments in form of a map") --- M src/sctp_proxy.erl 1 file changed, 7 insertions(+), 4 deletions(-)
Approvals: pespin: Looks good to me, approved laforge: Looks good to me, but someone else must approve Jenkins Builder: Verified
diff --git a/src/sctp_proxy.erl b/src/sctp_proxy.erl index e8460a4..badadcd 100644 --- a/src/sctp_proxy.erl +++ b/src/sctp_proxy.erl @@ -205,10 +205,13 @@
%% Event handler for all states handle_event(State, {call, From}, fetch_info, - #{handler := Pid} = S0) -> - ERABs = s1ap_proxy:fetch_erab_list(Pid), - S1 = maps:with([enb_aid, mme_aid, mme_addr, mme_port], S0), - S2 = S1#{state => State, erab_list => ERABs}, + #{handler := Pid, conn_cfg := ConnCfg} = S0) -> + ERABs = s1ap_proxy:fetch_erab_list(Pid), %% TODO: separate API + S1 = maps:with([enb_aid, mme_aid], S0), + S2 = S1#{state => State, + erab_list => ERABs, + mme_addr => maps:get(raddr, ConnCfg), + mme_port => maps:get(rport, ConnCfg)}, Reply = proplists:from_map(S2), {keep_state_and_data, {reply, From, Reply}};