Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

M1 - Consumption Reporting Provisioning API - Wrong response code for POST operation #156

Closed
dsilhavy opened this issue Apr 26, 2024 · 1 comment · Fixed by #160
Closed

M1 - Consumption Reporting Provisioning API - Wrong response code for POST operation #156

dsilhavy opened this issue Apr 26, 2024 · 1 comment · Fixed by #160
Assignees
@rjb1000 @davidjwbbc @stojkovicv
Labels
bug Something isn't working
Milestone
Version 1.4.1

Comments

@dsilhavy
Copy link
Contributor

Describe the bug

TS26512_M1_ConsumptionReportingProvisioning.yaml v 17.7.0 defines the response code for the path /provisioning-sessions/{provisioningSessionId}/consumption-reporting-configuration and the postoperation. According to the YAML file the response code shall be 201.

Our implementation currently returns code 204:

Bildschirmfoto 2024-04-26 um 09 41 28
@dsilhavy dsilhavy added the bug Something isn't working label Apr 26, 2024
@rjb1000
Copy link
Contributor

rjb1000 commented Apr 26, 2024

Agree with your analysis, @dsilhavy. The success response code for the activateConsumptionReporting operation should be 201 Created per the YAML.

If @davidjwbbc can reproduce it, we should fix it.

@davidjwbbc davidjwbbc added this to the Version 1.4.1 milestone Apr 26, 2024
davidjwbbc pushed a commit to davidjwbbc/rt-5gms-application-function that referenced this issue Apr 26, 2024
Bug-fix: Incorrect return for consumption report configuration creation
1297be0 
Fixes 5G-MAG#156, 5G-MAG#157.
@davidjwbbc davidjwbbc mentioned this issue Apr 26, 2024
Merged
@davidjwbbc davidjwbbc linked a pull request Apr 26, 2024 that will close this issue
Bug-fix: Response message for activateConsumptionReporting operation at M1 #160
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rjb1000 rjb1000

@davidjwbbc davidjwbbc

@stojkovicv stojkovicv

Labels
bug Something isn't working
Projects
5GMS: Consumption collection and reporting feature
Status: Done
Milestone
Version 1.4.1
4 participants
@dsilhavy @rjb1000 @davidjwbbc @stojkovicv