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

Stub for security, interoperability and fragment identifier considera… #30

Merged
merged 2 commits into from
Jun 29, 2022

Conversation

ioggstream
Copy link
Contributor

@ioggstream ioggstream commented Jun 23, 2022

This PR

  • adds stubs for the various consideration sections.

Note

  • the text is just a placeholder

Preview | Diff

spec/index.html Outdated Show resolved Hide resolved
spec/index.html Outdated Show resolved Hide resolved
spec/index.html Outdated Show resolved Hide resolved
Co-authored-by: Ted Thibodeau Jr <tthibodeau@openlinksw.com>
<h2>Fragment identifiers</h2>

<p>
YAML media type support both alias nodes and JSON Pointers [[RFC6905]]
Copy link
Contributor Author

@ioggstream ioggstream Jun 24, 2022

Choose a reason for hiding this comment

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

Note: here we could

  • import application/yaml fragid considerations
  • replace the text below after investigating yaml-ld specificities. See Fragment identifier #31

Copy link
Member

Choose a reason for hiding this comment

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

Again, JSON-LD puts this in IANA Considerations and describes it as being equivalent to other RDF syntaxes. The closest analogy is probably with XML and RDFa, where there is overlap between markup identifiers and fragment identifiers.

Note JSON-LD does not have any direct relationship to JSON Pointers.

We do need to consider and reconcile the overlap between the different YAML fragment identifiers, and the use of fragments in RDF, in general, which do not necessarily require that a fragment be directly associated with a marker in the document.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@gkellogg my proposal is

1- let's write down the stuff in these sections as specified in https://datatracker.ietf.org/doc/html/rfc8126#section-1.1
2- if the text just results in a series of references to external documents, we can just reference things into the IANA considerations
3- if the text is longer, we reference the document section

WDYT?

@gkellogg
Copy link
Member

We need to reconcile with #27. Maybe merge that first and then apply any updates after the fact. Or, this could be a PR into the branch for #27.

@gkellogg
Copy link
Member

(Assuming there’s overlap)

@ioggstream
Copy link
Contributor Author

I'll rebase when needed :) In the meanwhile let's talk on the content.

Copy link
Member

@gkellogg gkellogg left a comment

Choose a reason for hiding this comment

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

Approved, but subject to further refinement and reconciliation.

<h2>Security Considerations</h2>

<p>See the YAML media type registration.</p>
</section>
Copy link
Member

Choose a reason for hiding this comment

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

This duplicates/will duplicate the section in #27 IANA Considerations. The JSON-LD spec has this section, but links a reference to its own sub-section within IANA Considerations. We should probably do the same.

See, Security Considerations in § A. IANA Considerations.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

IANA Security considerations should reference this section.

Copy link
Member

Choose a reason for hiding this comment

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

IANA Security considerations should reference this section.

That makes it inconsistent with how it's done in JSON-LD. That may be fine if we're internally consistent, but it is different.

Copy link
Contributor Author

@ioggstream ioggstream Jun 25, 2022

Choose a reason for hiding this comment

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

In the past I have been pointed to https://datatracker.ietf.org/doc/html/rfc8126#section-1.1

The purpose of having a dedicated IANA Considerations section is to
provide a single place to collect clear and concise information and
instructions for IANA. Technical documentation should reside in
other parts of the document; the IANA Considerations should refer to
these other sections by reference only (as needed).

For example, see https://datatracker.ietf.org/doc/html/draft-ietf-httpbis-binary-message-05#section-7 .

Copy link
Member

Choose a reason for hiding this comment

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

Okay, wish we had known that when we did the JSON-LD IANA considerations, of course, that was way back in the 1.0 timeframe in 2014, so we should go with your division. We'll need to follow up after both PRs are merged. (I usually wait a week for comments to come in before merging with the consensus opinions).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Honestly, I didn't know that either:) that's all stuff I learned thanks to the reviews of the httpwg folks :)))

<p>
For general interoperability considerations on the serialization of
JSON documents in YAML, see [[YAML]].
</p>
Copy link
Member

Choose a reason for hiding this comment

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

Either treat the same as security considerations, and reference the sub-section in IANA Considerations, have that reference this part, or duplicate content.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

See above comment from RFC8126

<h2>Fragment identifiers</h2>

<p>
YAML media type support both alias nodes and JSON Pointers [[RFC6905]]
Copy link
Member

Choose a reason for hiding this comment

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

Again, JSON-LD puts this in IANA Considerations and describes it as being equivalent to other RDF syntaxes. The closest analogy is probably with XML and RDFa, where there is overlap between markup identifiers and fragment identifiers.

Note JSON-LD does not have any direct relationship to JSON Pointers.

We do need to consider and reconcile the overlap between the different YAML fragment identifiers, and the use of fragments in RDF, in general, which do not necessarily require that a fragment be directly associated with a marker in the document.

@gkellogg gkellogg added the spec Issue on specification label Jun 24, 2022
@gkellogg gkellogg merged commit 58a20c3 into main Jun 29, 2022
@gkellogg gkellogg deleted the ioggstream-28 branch June 29, 2022 21:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
spec Issue on specification
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants