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

Broken reference to rfc 7230 #2595

Closed
mattgarrish opened this issue Dec 5, 2023 · 3 comments · Fixed by #2597
Closed

Broken reference to rfc 7230 #2595

mattgarrish opened this issue Dec 5, 2023 · 3 comments · Fixed by #2597
Labels
Change-Class-2 Requested changes are of class 2 (per process) Editorial For classifying errata only. Use Type-Editorial for issues Errata Item accepted as an official errata for one or several specs Spec-EPUB3 The issue affects the core EPUB 3.3 Recommendation Spec-ReadingSystems The issue affects the EPUB Reading Systems 3.3 Recommendation Topic-PackageDoc The issue affects package documents Type-Editorial The issue does not affect conformance

Comments

@mattgarrish
Copy link
Member

Section 3.6 has a broken reference to RFC 7230:

When resources have to be located outside the EPUB container, EPUB creators are RECOMMENDED to reference them via the secure https URI scheme [rfc7230]

Looks like double brackets weren't used on this one.

(The use of URI looks fine here since it's defined in the rfc.)
@mattgarrish mattgarrish added Topic-PackageDoc The issue affects package documents Spec-EPUB3 The issue affects the core EPUB 3.3 Recommendation Type-Editorial The issue does not affect conformance ErratumRaised Raise an erratum, not yet accepted by the maintainers. Change-Class-2 Requested changes are of class 2 (per process) labels Dec 5, 2023
@iherman
Copy link
Member

iherman commented Dec 5, 2023

Sigh... :-)

@iherman iherman mentioned this issue Dec 6, 2023
Closed
1 task
@iherman
Copy link
Member

iherman commented Dec 6, 2023

Actually, it should be replaced by 9110 (see also #2596)

@iherman iherman added Errata Item accepted as an official errata for one or several specs and removed ErratumRaised Raise an erratum, not yet accepted by the maintainers. labels Dec 6, 2023
@iherman
Copy link
Member

iherman commented Dec 6, 2023

The reference should also be changed in the RS (see https://www.w3.org/TR/epub-rs-33/#sec-epub-rs-conf-remote-res)

@iherman iherman added the Spec-ReadingSystems The issue affects the EPUB Reading Systems 3.3 Recommendation label Dec 6, 2023
@mattgarrish mattgarrish mentioned this issue Dec 6, 2023
Merged
@mattgarrish mattgarrish added the Editorial For classifying errata only. Use Type-Editorial for issues label Dec 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Change-Class-2 Requested changes are of class 2 (per process) Editorial For classifying errata only. Use Type-Editorial for issues Errata Item accepted as an official errata for one or several specs Spec-EPUB3 The issue affects the core EPUB 3.3 Recommendation Spec-ReadingSystems The issue affects the EPUB Reading Systems 3.3 Recommendation Topic-PackageDoc The issue affects package documents Type-Editorial The issue does not affect conformance
Projects
Publishing Maintenance WG Project Tracker
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Replace references to obsolete rfc7230 with rfc9110 w3c/epub-specs
2 participants
@iherman @mattgarrish