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

Error showing HTTPS URL instead of internal path #1542

Open
SimonPRH opened this issue Aug 31, 2023 · 1 comment
Open

Error showing HTTPS URL instead of internal path #1542

SimonPRH opened this issue Aug 31, 2023 · 1 comment
Assignees
@rdeltour
Labels
priority: medium To be processed and published in one of the upcoming releases status: has PR The issue is being processed in a pull request type: regression Something that broke
Milestone
Next maintenance ...

Comments

@SimonPRH
Copy link

SimonPRH commented Aug 31, 2023 •
edited
Loading

I'm remediating some VERY old EPUB files, and got this error:

ERROR | OPF-032 | /content.opf (line 64, col 77) | Guide references "https://2694f716-dddd-4b2b-bbfe-64990fe68f69.epubcheck.w3c.org/OEBPS/images/cover.jpg" which is not a valid "OPS Content Document".

Obviously I'll change the guide item target to the cover XHTML page, but I'm sure it's not intended behaviour that this shows an HTTPS URL path. This happens in EPUB Check 5.0.1 and 5.1.1, I believe the error first appeared in the former. Seen this on multiple files now, though this particular file was created 13 years ago.
@rdeltour
Copy link
Member

This looks like a bug. The absolute URL is the internal representation of the resource used by EPUBCheck (since v5.0.0) and should not be presented to the user.
Thanks for the report! 👍

@rdeltour rdeltour self-assigned this Aug 31, 2023
@rdeltour rdeltour added status: accepted Ready to be further processed priority: medium To be processed and published in one of the upcoming releases type: regression Something that broke labels Aug 31, 2023
@rdeltour rdeltour added this to the Next maintenance release milestone Aug 31, 2023
@rdeltour rdeltour added status: in progress The issue is being implemented by the development team and removed status: accepted Ready to be further processed labels Dec 22, 2024
rdeltour added a commit that referenced this issue Dec 23, 2024
@rdeltour
fix: prevent some internal URLs from leaking in messages
e1871e2 
Fix #1542
@rdeltour rdeltour mentioned this issue Dec 23, 2024
Open
@rdeltour rdeltour added status: has PR The issue is being processed in a pull request and removed status: in progress The issue is being implemented by the development team labels Dec 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@rdeltour rdeltour

Labels
priority: medium To be processed and published in one of the upcoming releases status: has PR The issue is being processed in a pull request type: regression Something that broke
Projects
None yet
Milestone
Next maintenance release
Development

No branches or pull requests
2 participants
@rdeltour @SimonPRH