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.

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

Feature: uplift to TS 26.512 v17.3.0 #26

Merged

Conversation

davidjwbbc
Copy link
Contributor

This adds support for the new Content Hosting Configuration structure defined in TS 26.512 v17.3.0.

Closes #19.

The Provisioning Session ID is now dynamic. See the log output for the provisioning session ID and expected distribution URLs.

This also fixes a few bugs:

  • Fix segfault when certificates JSON file or ContentHostingConfiguration file cannot be opened.
  • Fix issue with certificates checks terminating early.
  • Fix errors generated when trying to duplicate unpopulated optional fields.
  • Fix copy of set.h in OpenAPI generator script to make the operation relative to the script location.
  • Change some log output to debug level instead of info. Use -m msaf -d on the command line to see the debug output.

@davidjwbbc davidjwbbc added bug Something isn't working enhancement New feature or request labels Dec 23, 2022
@davidjwbbc davidjwbbc self-assigned this Dec 23, 2022
@davidjwbbc
Copy link
Contributor Author

Note this PR requires PRs 5G-MAG/rt-common-shared#15 and 5G-MAG/rt-5gms-application-server#51 in order to work.

@davidjwbbc davidjwbbc added this to the MWC23 milestone Dec 23, 2022
examples/README.md Outdated Show resolved Hide resolved
src/5gmsaf/application-server.c Outdated Show resolved Hide resolved
src/5gmsaf/media-player.c Outdated Show resolved Hide resolved
Copy link
Contributor

@rjb1000 rjb1000 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for addressing my comments. Looks good now.

@davidjwbbc davidjwbbc merged commit 77eba59 into 5G-MAG:development Jan 18, 2023
@davidjwbbc davidjwbbc deleted the feature/uplift-to-ts26-512-v17-3-0 branch January 18, 2023 11:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Development

Successfully merging this pull request may close these issues.

3 participants