laforge submitted this change.

View Change

Approvals: laforge: Looks good to me, approved Jenkins Builder: Verified pespin: Looks good to me, but someone else must approve
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(-)

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 change 40904. To unsubscribe, or for help writing mail filters, visit settings.

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 <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>