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

Use of path part of URL in IngestConfiguration.entryPoint for pull ingest #19

Closed
davidjwbbc opened this issue Sep 13, 2022 · 8 comments
Closed
Assignees
Labels
3GPP Rel-16 Issues relating to 3GPP Release 16 specifications. 3GPP Rel-17 Issues relating to 3GPP Release 17 specifications. 3GPP TS 26.512 Issues relating to SA4's "5G Media Streaming (5GMS); Protocols" specification. 5GMS Content Hosting Adopted Clarification

Comments

@davidjwbbc
Copy link

Problem Description

In TS 26.512:

  • Table 7.6.3.1-1 states that for pull content the IngestConfiguration.entryPoint is the location from which content is pulled and is used as a base URL for requests for the pull ingest.
  • Section 8.2 states that for pull ingest "The IngestConfiguration.entryPoint shall not contain a path part."
  • Section 8.2 also states that pull ingest request URL is made up from a concatenation of the IngestConfiguration.entryPoint and the mappedPath, and does not include the IngestConfiguration.path.

The limitation in section 8.2 seems to go against the spirit of using the IngestConfiguration.entryPoint as a base URL by forbidding the use of the path part of the URL. By also excluding the IngestConfiguration.path from the method of creating the pull ingest URL, this forces any path required at the IngestConfiguration.entryPoint to be inserted using the Regex backed DistributionConfigurations.PathRewriteRules, when a computationally less expensive concatenation would have sufficed.

There should also be clarification about what is meant by "base URL" in table 7.6.3.1-1 as the simple meaning of it being a base prefix, as suggested by the description in section 8.2, is different from the interpretation and usage in URL resolution of a base URI according to IETF RFC3986.

Discussion

Should the restriction on the path part of the URL be dropped in Section 8.2 to allow simpler configurations to be possible when creating a Content Hosting Configuration?

Is the intention of "base URL" in table 7.6.3.1-1 to resolve URLs according to the usage of base URIs as stated in IETF RFC 3986?

@davidjwbbc davidjwbbc added Clarification 3GPP Rel-17 Issues relating to 3GPP Release 17 specifications. 3GPP TS 26.512 Issues relating to SA4's "5G Media Streaming (5GMS); Protocols" specification. labels Sep 13, 2022
@davidjwbbc davidjwbbc added this to the 3GPP SA4#121 milestone Sep 13, 2022
@rjb1000 rjb1000 added the 3GPP Rel-16 Issues relating to 3GPP Release 16 specifications. label Sep 13, 2022
@davidjwbbc
Copy link
Author

Proposal is to delete the sentence from 8.2 ("The IngestConfiguration.entryPoint shall not contain a path part.") to relax the rules for the entryPoint field.

@rjb1000 to raise a draft CR to stimulate discussion in SA4.

@rjb1000 rjb1000 added the Adopted label Oct 4, 2022
@rjb1000
Copy link
Contributor

rjb1000 commented Oct 6, 2022

3GPP draft Change Request to TS 26.512 Rel-16 S4aI221386 opened for initial discussion at today's 3GPP SA4 MBS subworking group ad hoc call. Discussion will continue on 20th October.

(Plan is to create a Rel-17 counterpart later that makes equivalent changes there.)

@rjb1000
Copy link
Contributor

rjb1000 commented Oct 26, 2022

Latest draft CR proposes deprecating the confusingly named IngestConfiguration.entryPoint and IngestConfiguration.path in favour of IngestConfiguration.baseURL.

To compensate, a ContentHostingConfiguration.entryPointPath is added, applicable to both ingest and distribution base URLs.

@rjb1000
Copy link
Contributor

rjb1000 commented Oct 26, 2022

Latest draft CRs available to 5G-MAG members on Causeway as CDT0107.3GPP SA4#121 TS26.512 API corrections.

@rjb1000
Copy link
Contributor

rjb1000 commented Nov 8, 2022

Formal CRs submitted to SA4#121:

@rjb1000
Copy link
Contributor

rjb1000 commented Nov 18, 2022

The following Change Requests were agreed at SA4#121 Closing Plenary yesterday afternoon:

  • TS 26.512:
    • Rel-16: CR0027r2 S4-221605.
    • Rel-17: CR0028r2 S4-221606.

@rjb1000
Copy link
Contributor

rjb1000 commented Dec 19, 2022

Change Requests approved at SA#98-e:

@rjb1000
Copy link
Contributor

rjb1000 commented Jan 9, 2023

New specifications published:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3GPP Rel-16 Issues relating to 3GPP Release 16 specifications. 3GPP Rel-17 Issues relating to 3GPP Release 17 specifications. 3GPP TS 26.512 Issues relating to SA4's "5G Media Streaming (5GMS); Protocols" specification. 5GMS Content Hosting Adopted Clarification
Projects
Development

No branches or pull requests

3 participants