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

fix: サマリ取得の動作改善+動作設定を可能にする #23

Merged
merged 1 commit into from
Mar 17, 2024

Conversation

samunohito
Copy link
Member

What

以下の対応をしました。

  • GETでリンク先を取得する前にHEADでリクエストし、その内容でcontent-typeやcontent-lengthを検証するようにした
    →冗長なbodyのダウンロードを抑止するようにした
  • 下記パラメータをライブラリの外から差し込めるようにした
    • userAgent →ライブラリがリクエスト時に名乗るUser-Agent
    • responseTimeout →リクエスト時の各プロセスの制限時間
    • operationTimeout →リクエスト~レスポンス受信完了までの制限時間
    • contentLengthLimit →この値よりも相手から帰ってきたcontent-lengthが大きい場合は失敗扱いにする
    • contentLengthRequired →相手がcontent-lengthを返さない場合は失敗扱いにする

Why

fix #22

Additional info (optional)

テストを追加して確認

@samunohito samunohito requested a review from syuilo March 15, 2024 06:16
@samunohito
Copy link
Member Author

もしこの対応がマージされたら、マイナーバージョンをあげてリリースしていただきたいです

@syuilo syuilo merged commit c261071 into misskey-dev:master Mar 17, 2024
2 checks passed
@syuilo
Copy link
Member

syuilo commented Mar 17, 2024

👍

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 this pull request may close these issues.

サマリ取得の動作改善+動作設定を可能にする
2 participants