build: 依存関係の更新 (2023-09-05)

[kou029w]

TODO

• build: 依存関係の更新 (2023-09-05)
• test: Define ts-jest config under globals is deprecated.
• Docker images
  • mariadb:10.11.2-jammy → 11.1.2-jammy
  • bitnami/moodle:4.1.2-debian-11-r6 → LTSのままにしておく
  • postgres:15.2-alpine3.17 → 15.4-alpine3.18
  • node:18.15.0-alpine3.17 → 18.17.1-alpine3.18
• GitHub Actions
  • actions/checkout@v3 → actions/checkout@v4
• Lint
  • components/organisms/BookForm.tsx
  • components/organisms/ContentPreview.stories.tsx:24
  • components/organisms/ContentPreview.stories.tsx:35
  • components/organisms/Video/VideoJs.tsx:60
  • components/organisms/Video/index.tsx:220
  • components/organisms/Video/index.tsx:279
  • components/templates/Books.tsx:91
  • components/templates/BooksImport.tsx:96
  • components/templates/DeepLinkBooks.tsx:95
  • components/templates/TopicImport.tsx:90
  • components/templates/Topics.tsx:159
  • pages/topics/edit.tsx:43
  • server/utils/zoom/api.ts:1
  • server/utils/zoom/api.ts:2
  • server/utils/zoom/import.ts:4
  • utils/eventLogger/playerTracker.ts:102
  • utils/eventLogger/playerTracker.ts:114
  • utils/playbackRate.ts:44
  • utils/useVolume.ts:61
  • utils/useVolume.ts:61
  • utils/videoJsDurationChangeShims.ts:18
  • utils/youtubePlayedShims.ts:16
  • utils/youtubePlayedShims.ts:38
  • utils/youtubePlayedShims.ts:38
  • utils/youtubePlayedShims.ts:41
  • utils/youtubePlayedShims.ts:41
  • utils/youtubePlayedShims.ts:46
  • utils/youtubePlayedShims.ts:46

[ties-makimura]

他のPRの変更とconflictする恐れがあるのでWIP

現時点での他PRのマージ完了

[kou029w]

$ eslint --ext .js,.jsx,.ts,.tsx .

/home/nebel/chibichilo/server/utils/zoom/api.ts
  1:17  error  Cannot find module 'got' or its corresponding type declarations  tsc/config
  2:29  error  Cannot find module 'got' or its corresponding type declarations  tsc/config

/home/nebel/chibichilo/server/utils/zoom/import.ts
  4:17  error  Cannot find module 'got' or its corresponding type declarations  tsc/config

使用しているパッケージ got に関する TypeScript 型検査でのエラー。
Got v13 の変更によって "module": "node16", "moduleResolution": "node16" へのコンパイルオプションの変更が必要。変更することによる影響範囲が広いので悩ましい。

sindresorhus/got@v12.6.1...v13.0.0#diff-7ae45ad102eab3b6d7e7896acd08c427a9b25b346470d7bc6507b6481575d519R10

• https://github.com/sindresorhus/got/releases

[kou029w]

Got v13 の変更によって "module": "node16", "moduleResolution": "node16" へのコンパイルオプションの変更が必要。

Gotを取り除き、代わりにfetch()を使うように書き換えました。

---

また、package.json のバージョンを、リリースに合わせて v2.5.0 に更新しました:

chibichilo/package.json

Line 3 in 43f7c16

"version": "2.5.0",

[ties-makimura]

• Zoomインポートを行うと，Zoomからのビデオ取得・ブックとトピックの作成され，機能自体は一見問題ないように見えるが，ログにSyntaxErrorが記録される．

2023-10-13 12:00:00 INFO begin zoom import... ZoomImportLog
2023-10-13 12:00:04 ERROR SyntaxError: Unexpected end of JSON input: email:*** ZoomImportLog
SyntaxError: Unexpected end of JSON input
    at JSON.parse (<anonymous>)
    at parseJSONFromBytes (/home/nodejs/chibichilo/server/dist/index.js:134145:19)
    at successSteps (/home/nodejs/chibichilo/server/dist/index.js:134119:27)
    at specConsumeBody (/home/nodejs/chibichilo/server/dist/index.js:134125:9)
    at _Response.json (/home/nodejs/chibichilo/server/dist/index.js:134015:18)
    at jsonFetcher (/home/nodejs/chibichilo/server/dist/index.js:161918:20)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async zoomRequest (/home/nodejs/chibichilo/server/dist/index.js:161950:10)
    at async ZoomImport.importBooks (/home/nodejs/chibichilo/server/dist/index.js:162052:9)
    at async zoomImport (/home/nodejs/chibichilo/server/dist/index.js:161993:5)
    at async Job.job (/home/nodejs/chibichilo/server/dist/index.js:162215:7)
2023-10-13 12:00:04 INFO end zoom import... ZoomImportLog

• Videojsプレイヤーにある早送りと巻き戻しのアイコンが●になっている．クリックすると機能自体は正常に動作する．
  • master ブランチ
    
  • update-2023-09-05 ブランチ
    

[kou029w]

確認ありがとうございます!

前者は 9d261e1#diff-7e10cf8f5de0e980a47897beee74486e28d43d4b9caf85bba4a3a5693b06a0b7L385 にあったcontent-typeヘッダーが抜け落ちている差分を見つけたのでこれは変更してヘッダーを与えてみてもう少し調査してみます

後者は予想外でしたがまず再現するか調査してみます

[kou029w]

SyntaxErrorの件はレコーディングの削除後のレスポンスが空のケースで動作上も問題なくcontent-typeは関係無さそうでした

        await zoomRequest(
          `/meetings/${deletemeeting}/recordings`,
          { action: "trash" },
          "DELETE"
        );

しかしエラーではないにもかかわらずエラーメッセージを出すのは運用上は適さないかと思うのでレコーディングの削除処理については Response.json() メソッドを呼ばない fetcher を使用するという対処が考えられます。

[kou029w]

videojs-seek-buttonsについては mister-ben/videojs-seek-buttons#130 に同様の課題を発見しました。
v4.0.3 アップデート後:

[kou029w]

しかしエラーではないにもかかわらずエラーメッセージを出すのは運用上は適さないかと思うのでレコーディングの削除処理については Response.json() メソッドを呼ばない fetcher を使用するという対処が考えられます。

JSONのパースを行わないリクエストを行うための zoomRequestWithoutParse() 関数を実装し、レコーディングの削除処理の中でこれを呼ぶように変更しました。

[ties-makimura]

• Zoomでエラーログ出ないことを確認

2023-10-18 17:45:00 INFO begin zoom import... ZoomImportLog
2023-10-18 17:45:04 INFO end zoom import... ZoomImportLog

• 早送りと巻き戻しのアイコンが直っていることを確認