Commit d0ddfa2d authored by Stefan Spettel's avatar Stefan Spettel

fix(smf): Send HTTP 201 only once for the failure case

Signed-off-by: default avatarStefan Spettel <stefan.spettel@phine.tech>
parent 074889ff
......@@ -1781,6 +1781,17 @@ void smf_context::handle_pdu_session_create_sm_context_request(
// TODO:
// un-subscribe to the modifications of Session Management Subscription data
// for (SUPI, DNN, S-NSSAI)
// Send HTTP reply to PDU Session Establishment Request
// TODO Stefan: I believe this is not standard-compliant, we should just
// send an error code here (see 29.502 Table 6.1.3.2.3.1-3)
pdu_session_create_sm_context_response sm_context_response = {};
sm_context_response.set_http_code(
http_status_code_e::HTTP_STATUS_CODE_201_CREATED);
smf_app_inst->trigger_session_create_sm_context_response(
sm_context_response, smreq->pid);
}
// Step 10. if error when establishing the pdu session,
......@@ -1812,16 +1823,6 @@ void smf_context::handle_pdu_session_create_sm_context_request(
// clear the created context??
// TODO:
// Send HTTP reply to PDU Session Establishment Request
// TODO Stefan: I believe this is not standard-compliant, we should just
// send an error code here (see 29.502 Table 6.1.3.2.3.1-3)
pdu_session_create_sm_context_response sm_context_response = {};
sm_context_response.set_http_code(
http_status_code_e::HTTP_STATUS_CODE_201_CREATED);
smf_app_inst->trigger_session_create_sm_context_response(
sm_context_response, smreq->pid);
// Create PDU Session Establishment Reject and embedded in
// Namf_Communication_N1N2MessageTransfer Request
Logger::smf_app().debug("Create PDU Session Establishment Reject");
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment