laforge has submitted this change. (
https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/40904?usp=email )
Change subject: sctp_proxy: return handler Pid in fetch_info/0
......................................................................
sctp_proxy: return handler Pid in fetch_info/0
Instead of returning the E-RAB list, give the caller access to
the associated s1ap_proxy process by returning its Pid. This
allows the caller to use API provided by the s1ap_proxy directly.
Change-Id: I3911bfc107fea3ca572cf44f1d93b768b3c82b4f
---
M src/sctp_proxy.erl
1 file changed, 1 insertion(+), 2 deletions(-)
Approvals:
laforge: Looks good to me, approved
Jenkins Builder: Verified
pespin: Looks good to me, but someone else must approve
diff --git a/src/sctp_proxy.erl b/src/sctp_proxy.erl
index badadcd..b9258c3 100644
--- a/src/sctp_proxy.erl
+++ b/src/sctp_proxy.erl
@@ -206,10 +206,9 @@
%% Event handler for all states
handle_event(State, {call, From}, fetch_info,
#{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,
+ handler => Pid,
mme_addr => maps:get(raddr, ConnCfg),
mme_port => maps:get(rport, ConnCfg)},
Reply = proplists:from_map(S2),
--
To view, visit https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/40904?usp=email
To unsubscribe, or for help writing mail filters, visit
https://gerrit.osmocom.org/settings?usp=email
Gerrit-MessageType: merged
Gerrit-Project: erlang/osmo-s1gw
Gerrit-Branch: master
Gerrit-Change-Id: I3911bfc107fea3ca572cf44f1d93b768b3c82b4f
Gerrit-Change-Number: 40904
Gerrit-PatchSet: 3
Gerrit-Owner: fixeria <vyanits...@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <lafo...@osmocom.org>
Gerrit-Reviewer: pespin <pes...@sysmocom.de>
