fixeria has submitted this change. ( 
https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/39110?usp=email )

Change subject: s1ap_proxy: add a TODO about an optional IE
......................................................................

s1ap_proxy: add a TODO about an optional IE

Change-Id: I3980ae86a61e59af5cec8e7138062cc72a6cd130
---
M src/s1ap_proxy.erl
1 file changed, 1 insertion(+), 0 deletions(-)

Approvals:
  pespin: Looks good to me, but someone else must approve
  laforge: Looks good to me, approved
  Jenkins Builder: Verified




diff --git a/src/s1ap_proxy.erl b/src/s1ap_proxy.erl
index 86c2bed..5cacb59 100644
--- a/src/s1ap_proxy.erl
+++ b/src/s1ap_proxy.erl
@@ -365,6 +365,7 @@
                                  value = C0} = Msg}, S0) ->
     ?LOG_DEBUG("Processing INITIAL CONTEXT SETUP RESPONSE"),
     s1gw_metrics:ctr_inc(?S1GW_CTR_S1AP_PROXY_IN_PKT_INIT_CTX_RSP),
+    %% TODO: handle optional E-RAB Failed to Setup List IE
     case handle_ies(C0#'InitialContextSetupResponse'.protocolIEs,
                     ?'id-E-RABSetupListCtxtSURes', S0) of
         {{ok, IEs}, S1} ->

--
To view, visit https://gerrit.osmocom.org/c/erlang/osmo-s1gw/+/39110?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: I3980ae86a61e59af5cec8e7138062cc72a6cd130
Gerrit-Change-Number: 39110
Gerrit-PatchSet: 1
Gerrit-Owner: fixeria <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <[email protected]>
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>

Reply via email to