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

The output file scheme is not supported #489

Closed
YellowPhoenix18 opened this issue Sep 10, 2023 · 5 comments · Fixed by #490
Closed

The output file scheme is not supported #489

YellowPhoenix18 opened this issue Sep 10, 2023 · 5 comments · Fixed by #490
Assignees

Comments

@YellowPhoenix18
Copy link

When use knp-snappy under windows you get a issue with the scheme when using getOutputFromHtml since Version 1.4.3.
In the background the parse_url parses the drive-letter as scheme, resulting in "c" as scheme and a error with
"The output file scheme is not supported. Expected '' or 'file' but got 'c'."

@alexpozzi
Copy link
Member

alexpozzi commented Sep 11, 2023

Hello,
I'm gonna work on a fix soon, in the meantime, you should use file:///c:\\your\\path\\... instead.
The fix will be available in the next Snappy release.

@YellowPhoenix18
Copy link
Author

Yeah, basically I would agree, the issue is, that getOutputHtml creates temp files automatically and these files are the issue. So the fix must be in the code itself. I see no option to fix this issue myself, as I cant change the scheme of the temp files.

@alexpozzi
Copy link
Member

True, I didn't think about that case. I'm afraid that your only option, at the moment, is to temporarily revert to 1.4.2.

@YellowPhoenix18
Copy link
Author

No worries. The issue is as far as I can see only under windows, as only windows is adding in the drive-letter. I tested the same code yesterday with 1.4.3 on Linux and there it was working fine, as a schema with no content is also allowed by your code.

For now I just uncommented the check-line in my local vendor-folder on windows, as this is only my dev-machine.

@xicond
Copy link

xicond commented Sep 12, 2023

Unfortunately file:/// is not working in the executable wkhtmltopdf

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants