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

Script elements must reference only local scripts #870

Closed
mattgarrish opened this issue Nov 12, 2018 · 1 comment
Closed

Script elements must reference only local scripts #870

mattgarrish opened this issue Nov 12, 2018 · 1 comment
Assignees
@rdeltour @GrayWolfMT
Labels
spec: EPUB 3.2 Impacting the support of EPUB 3.2 status: has PR The issue is being processed in a pull request
Milestone
4.2.0-RC

Comments

@mattgarrish
Copy link
Member

HTML and SVG script elements must not reference remote resources from their href/src attributes.

Reference: https://w3c.github.io/publ-epub-revision/epub32/spec/epub-spec.html#sec-resource-locations-script
@mattgarrish mattgarrish added the spec: EPUB 3.2 Impacting the support of EPUB 3.2 label Nov 12, 2018
@mattgarrish mattgarrish changed the title Scripts element must reference only local scripts Script elements must reference only local scripts Nov 12, 2018
@mattgarrish mattgarrish added the status: accepted Ready to be further processed label Nov 13, 2018
@GrayWolfMT GrayWolfMT self-assigned this Jan 27, 2019
@rdeltour
Copy link
Member

Depends on the resolution of w3c/publ-epub-revision#1225 and w3c/publ-epub-revision#1234

@rdeltour rdeltour added status: blocked Blocked by another issue or situation and removed status: accepted Ready to be further processed labels Feb 25, 2019
@rdeltour rdeltour added this to the 4.2.0-RC milestone Feb 25, 2019
@rdeltour rdeltour added the type: spec The issue is related to a Specification update label Mar 12, 2019
rdeltour added a commit that referenced this issue Mar 12, 2019
@rdeltour
test: test that remote scripts are disallowed
61fdfdc 
Close #870
@rdeltour rdeltour mentioned this issue Mar 12, 2019
Merged
rdeltour added a commit that referenced this issue Mar 12, 2019
@rdeltour
test: test that remote scripts are disallowed
25c8449 
Close #870
@rdeltour rdeltour added status: has PR The issue is being processed in a pull request and removed status: blocked Blocked by another issue or situation type: spec The issue is related to a Specification update labels Mar 12, 2019
@rdeltour rdeltour self-assigned this Mar 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rdeltour rdeltour

@GrayWolfMT GrayWolfMT

Labels
spec: EPUB 3.2 Impacting the support of EPUB 3.2 status: has PR The issue is being processed in a pull request
Projects
None yet
Milestone
4.2.0-RC
Development

No branches or pull requests
3 participants
@rdeltour @mattgarrish @GrayWolfMT