-
-
Notifications
You must be signed in to change notification settings - Fork 24
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
feat: support ignoreHTTPSErrors when Playwright browser opens a new page #531
feat: support ignoreHTTPSErrors when Playwright browser opens a new page #531
Conversation
@Mura-Mi PRありがとうございます! 以下の点をご確認お願いします
すみません、こちらの問題は修正しましたので現時点でのmain branchをrebaseしてご使用ください。 ref: #532 |
@spring-raining 確認ありがとうございます。 desc の記載、rebase なども対応します 👍 |
d7b104f
to
23ed255
Compare
@spring-raining 検証手順の記載、プロキシ設定の追加、ブランチのりベースを行いました。 ご確認をお願いします! |
src/commands/build.ts
Outdated
@@ -7,6 +7,7 @@ try { | |||
const program = setupBuildParserProgram(); | |||
program.parse(process.argv); | |||
const options = program.opts(); | |||
console.log(options); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
こちらの行の削除をお願いします!
console.log(options); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
失礼しました!削除したうえでプロキシ対応のコミットをsquashしてあります。
23ed255
to
c8f2c69
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
良さそうです。ありがとうございます!
about
fix #530
Playwright の Browser が
newPage
する際のオプションに ignoreHTTPSErrors をCLIフラグから設定できるようにしました。環境依存の属性なので build コマンド及び preview コマンドの CLI オプションのみから設定できるものとし、 vivliostyle.config.js の設定項目には加えていません。
動作確認
パブリックインターネットへのアクセスに自己署名証明書を利用する必要のある環境にて、
--ignore-https-errors
オプションを付して数式記法を含む markdown ドキュメントをビルドし、レンダリング結果に mathjax 記法による数式レンダリングが正常になされていることを確認。