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

PDFが出力できない #91

Closed
Tracked by #89
ogwata opened this issue Aug 27, 2021 · 36 comments
Closed
Tracked by #89

PDFが出力できない #91

ogwata opened this issue Aug 27, 2021 · 36 comments
Assignees

Comments

@ogwata
Copy link
Member

ogwata commented Aug 27, 2021

「Action」メニューから「PDF」を選択すると、画面下部に「Build Started」という緑色のメッセージが表示され、「Action」メニュー左側にアニメーションで回転が表示されます。しかし、その後10分たってもPDFが生成されません。

動画を撮ってみました(容量制限のため8分でカットしてありますが、元ファイルは10分あります)。

2021-08-28.0.22.16-480p-b.mov
@ogwata
Copy link
Member Author

ogwata commented Aug 28, 2021

@takanakahiko
fix #91 のコミットありがとうございます!
ただ、私の環境では症状は全く変わらず、やはりPDFは出力されないままです。再度ご確認いただけないでしょうか。

@ogwata ogwata reopened this Aug 28, 2021
@takanakahiko
Copy link
Member

ですね、こっちでも直っていませんでした。引き続き調査してみます。

@ogwata
Copy link
Member Author

ogwata commented Aug 28, 2021

ただ、#90の404エラーは出なくなったようです!

@takanakahiko
Copy link
Member

今月中にやりたい

@takanakahiko takanakahiko self-assigned this Oct 25, 2021
@ogwata
Copy link
Member Author

ogwata commented Nov 3, 2021

ご苦労さまです。ただ、確認してみましたが私の環境では出力できませんでした。
動画を撮影してみましたので、ご確認ください。

2021-11-04.7.57.18.mov

@ogwata ogwata reopened this Nov 3, 2021
@takanakahiko
Copy link
Member

なるほど、週末に確認してみますね。

@ogwata
Copy link
Member Author

ogwata commented Nov 5, 2021

ログインからPDF出力までをDevToolで見てみました。
ご参考まで

2021-11-05.21.46.25.mov

@takanakahiko
Copy link
Member

数千ページになるようなプロジェクトでない限りは成功するようになりました!
ご報告&デバック協力ありがとうございました!

@takanakahiko
Copy link
Member

意図したレイアウトにならない問題はありそうですがその場合は別途Issueを建てましょう。
このIssueは一旦閉じますね。

@MurakamiShinyu
Copy link
Member

私の方で今また試しましたが、PDF出力できません。reopenします。
「Your build has been started」のメッセージが出たあと、いつまで待っても処理中のぐるぐるが回ったままになります。

Screen Shot 2022-01-28 at 13 38 33

@AyumuTakai
Copy link
Contributor

Slackのこちらのスレッドでも話を進めていますが、作業待ちです。

https://vivliostyle.slack.com/archives/CR14JBXCJ/p1640536666190400

@ogwata
Copy link
Member Author

ogwata commented Jan 28, 2022

Slackのこちらのスレッドでも話を進めていますが、作業待ちです。

https://vivliostyle.slack.com/archives/CR14JBXCJ/p1640536666190400

了解いたしました。
このIssueが解決できればアルファ版がリリースできます。
@takanakahiko さん、見通しをご報告いただけませんか。

@takanakahiko
Copy link
Member

お待たせして申し訳ございません。
こちらですが、一部環境では上手く動いているような感じでして調査に難航しております。
また明日の朝と夜にも調べてみるのでもう少しお待ちください🙇‍♂️

@takanakahiko
Copy link
Member

遅くなりましたが修正しました〜。
権限管理( IAM )に不適切なポイントがありうまく動いていなかったようです。

@ogwata さんにチェックしていただき問題なければ閉じていただければと。

@ogwata
Copy link
Member Author

ogwata commented Mar 4, 2022

コミットありがとうございます!
ただ、1/28 @MurakamiShinyu さんのコメントと同じ状況のようで、
Buildがスタートした後、いつまでも処理中のままです。
動画を撮ってみました。ご確認ください。

2022-03-05.8.38.58.mov

@AyumuTakai
Copy link
Contributor

https://vivliostyle-pub-develop.vercel.app/ で試したところ、私はPDF生成に成功しました。

何か条件が違うのでしょうか。

@ogwata
Copy link
Member Author

ogwata commented Mar 5, 2022

@AyumuTakai
コメントと検証、ありがとうございます。
こちらも同じURLです(ちなみに、まだグルグル回ってます)。
https://vivliostyle-pub-develop.vercel.app/

スクリーンショットで、なにか他の違いが分かるでしょうか?

@AyumuTakai
Copy link
Contributor

AyumuTakai commented Mar 5, 2022

動画見るとあまり条件変わらないですね。

私が以前試したときにPubのプレビューは表示されるけれど、vivliostyle.conf.jsのエラーでローカルのCLIでビルドは通らないケースがありました。
現在のPubのプレビューはvivliostyle.conf.jsの設定をthemeプロパティ以外使わないので、CLIと動作が違います。
もしかしたらDocker上のVivliostyle-CLIでエラーになっているのかも。

@ogwata
Copy link
Member Author

ogwata commented Mar 5, 2022

Chromeの更新がきていたので再起動してやり直しましたが、やはり同様の症状です。
devtoolのスクリーンショットは以下の通りです。
スクリーンショット 2022-03-05 10 10 53

@AyumuTakai
Copy link
Contributor

動画ではcross-siteのエラーは出ていないようですが、その後のスクリーンショットでは発生しているようです。
このエラーがどのタイミングで出るか、devtoolsを表示してからPDFエクスポートを試していただけませんか?
ビルド開始と同時に出るのか、暫くしてから出るのか確認したいです。

私の環境では、成功するリポジトリ、グルグルのままのリポジトリ双方でこのエラーは出ていません。

@ogwata
Copy link
Member Author

ogwata commented Mar 5, 2022

ご指示のように最初からDevToolを立ち上げて動画を撮影してみましたが、今度は同じcross-siteのエラーがでません。
なお、この動画撮影の終了後(今も処理中)も、同じエラーが出ないままです。

2022-03-05.10.43.35.mov

@ogwata
Copy link
Member Author

ogwata commented Mar 5, 2022

ちなみに、現在のdevtoolのスクリーンショットです。(動画の終了後、少しエラーが増えた)
スクリーンショット 2022-03-05 11 13 11

@AyumuTakai
Copy link
Contributor

検証ありがとうございます。
cross-site関連は何か別の理由によるものではないかと思っていましたが、間違い無さそうです。

その他のエラーはReactの更新エラーのたぐいだと思いますが、エラー内容は表示されないでしょうか。

他に考えられる原因として、私のPubのアカウントはここ一ヶ月の間に一度削除して作りなおしています。
アカウントの削除機能は今のところ無いので、他の処理に割り込ませる形でfirebaseのAPIを操作して無理矢理実行しました。

PDFエクスポートに成功している @takanakahiko さんがPDFエクスポート機能を実装しているときに御自身のアカウントを作り直したりしていたらIAMの設定変更が反映されているか否かかもしれません。

@MurakamiShinyu
Copy link
Member

私もテストしてみました。「ビルドを開始しました」でグルグル回ったままです。動画:

Screen.Recording.2022-03-05.at.11.12.49.mov

@takanakahiko
Copy link
Member

MurakamiShinyu/pubtest2 のビルドログを見てたんですが、 Error: No input is set. Please set an appropriate entry or a Vivliostyle config file. と出ていたので、 config 辺りに問題がありそうですかね?引き続き調査します。

@takanakahiko
Copy link
Member

御自身のアカウントを作り直したりしていたらIAMの設定変更が反映されているか否かかもしれません。

これは違いますね。
CloudRun に付与する権限を変えた感じですので、個人のアカウントの作り直しが IAM に影響することはないはずです。

@AyumuTakai
Copy link
Contributor

AyumuTakai commented Mar 5, 2022

MurakamiShinyu/pubtest2 には vivliostyle.config.js が存在しないのでそれが原因だと思います。
vivliostyle.config.jsが無い場合や内容が処理できない場合にはエクスポート機能を実行しないなどの対応が必要ですね。
CLIのバージョンと整合性を取るのが大変そうですが。

CloudRunの権限について承知しました。他の原因を探ってみます。

@MurakamiShinyu
Copy link
Member

MurakamiShinyu/pubtest2 には vivliostyle.config.js が存在しないのでそれが原因だと思います。

そうでした。vivliostyle.config.js が存在するリポジトリでテストしたところ、PDF生成に成功しました。

Screen.Recording.2022-03-05.at.12.17.46.mov

@AyumuTakai
Copy link
Contributor

この制限はドキュメント等に書いていても嵌るユーザが出そうなのでvivliostyle.config.jsの存在チェックとCLIのinit相当の機能を実装してみます。

@ogwata
Copy link
Member Author

ogwata commented Mar 5, 2022

この制限はドキュメント等に書いていても嵌るユーザが出そうなのでvivliostyle.config.jsの存在チェックとCLIのinit相当の機能を実装してみます。

ありがとうございます。
ただ、私のmybookリポジトリにはvivliostyle.config.js はあるので、上記実装だけでは問題を解決できませんね……。

@AyumuTakai
Copy link
Contributor

AyumuTakai commented Mar 5, 2022

mybookリポジトリをクローンしてローカルのCLIで実行したところ、section-1.mdが無いというエラーになりました。

✖ Error: ENOENT: no such file or directory, open '/Users/takaiayumu/working/ogwata/mybook/section-1.md'

やっぱりvivliostyle.config.jsのチェック機能は必要ですね...。
CLIと共通にしたいので、ここもライブラリ化できるとうれしいのですが。

もしくはDockerのstdoutとstderrをfirestoreに書き出してもらって、クライアント側でエラーハンドリングするか。

@ogwata
Copy link
Member Author

ogwata commented Mar 5, 2022

mybookリポジトリをクローンしてローカルのCLIで実行したところ、section-1.mdが無いというエラーになりました。

なるほど、vivliostyle.config.js を修正して、出力してみます!

@ogwata
Copy link
Member Author

ogwata commented Mar 5, 2022

vivliostyle.config.js からsection-1.mdの指定を削除して出力してみたところ、無事にPDFが出力できました!
私の認識が間違っていなければ、これでアルファ版のレベルに達したことになります。
開発者の皆さんに、心から感謝申し上げます
スクリーンショット 2022-03-05 13 41 10

@ogwata
Copy link
Member Author

ogwata commented Mar 5, 2022

本日の開発者会議で了解が取れたので、本Issueはクローズします。

@ogwata ogwata closed this as completed Mar 5, 2022
@ogwata
Copy link
Member Author

ogwata commented Mar 8, 2022

本Issueについて、march2022開発者会議で、@takanakahikoさんは以下のようにコメントされていました。

ただ出力される PDF にテーマが適用されていないのは問題かなと思っています。今はそれに取り組んでいます。これは遅くとも1週間程度で終わらせたいのですが、少なくともこれが終わるまでアルファ版リリースは待っていただきたいです。

また、再テストしたところ、出力されるPDFには以下のような問題があるようです。

  1. 選択されているファイルだけでなく、リポジトリにあるすべてのMDファイルがPDFに出力されてしまう
  2. MDファイルにscript要素でWebフォントを埋め込んだ場合、PDFの内容が乱れる

上記のテストは、以下の2つのリポジトリでおこないました。

なお、上記2の症状は、mybook-2のMDファイルで確認し、出力されたPDFは下記にあります。

以上の理由で、本Issueを再オープンします。

@ogwata ogwata reopened this Mar 8, 2022
@MurakamiShinyu
Copy link
Member

@ogwata

以上の理由で、本Issueを再オープンします。

このissueを再オープンではなく、別issueにするべきでしょう。

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

No branches or pull requests

4 participants