pespin has submitted this change. ( https://gerrit.osmocom.org/c/erlang/osmo-epdg/+/35976?usp=email ) Change subject: Send correct APN in GSUP CEAI Tunnel Response ...................................................................... Send correct APN in GSUP CEAI Tunnel Response Change-Id: Ifbd78720436690e9ee40688e455ab61d65085738 --- M src/epdg_gtpc_s2b.erl M src/gsup_server.erl 2 files changed, 12 insertions(+), 2 deletions(-) Approvals: pespin: Looks good to me, approved Jenkins Builder: Verified diff --git a/src/epdg_gtpc_s2b.erl b/src/epdg_gtpc_s2b.erl index 9b6d9ed..309ed79 100644 --- a/src/epdg_gtpc_s2b.erl +++ b/src/epdg_gtpc_s2b.erl @@ -386,6 +386,7 @@ lager:info("s2b: Updated Session after create_session_response: ~p~n", [Sess2]), State1 = update_gtp_session(Sess0, Sess2, State0), ResInfo = #{ + apn => binary_to_list(Sess0#gtp_session.apn), eua => conv:gtp2_paa_to_epdg_eua(Paa), local_teid => Bearer#gtp_bearer.local_data_tei, remote_teid => RemoteTEID, diff --git a/src/gsup_server.erl b/src/gsup_server.erl index 6930673..6bb98ee 100644 --- a/src/gsup_server.erl +++ b/src/gsup_server.erl @@ -156,10 +156,10 @@ lager:info("tunnel_response for ~p: ~p~n", [Imsi, Result]), Socket = State#gsups_state.socket, case Result of - {ok, #{eua := Eua}} -> + {ok, #{apn := Apn, eua := Eua}} -> PdpInfo = #{pdp_context_id => 0, pdp_address => conv:epdg_eua_to_gsup_pdp_address(Eua), - access_point_name => "foobar.apn", + access_point_name => Apn, quality_of_service => <<0, 0, 0>>, pdp_charging => 0}, Resp = #{message_type => epdg_tunnel_result, -- To view, visit https://gerrit.osmocom.org/c/erlang/osmo-epdg/+/35976?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: Ifbd78720436690e9ee40688e455ab61d65085738 Gerrit-Change-Number: 35976 Gerrit-PatchSet: 1 Gerrit-Owner: pespin &lt;pespin(a)sysmocom.de&gt; Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: pespin &lt;pespin(a)sysmocom.de&gt; Gerrit-MessageType: merged