misskey-dev / misskey

🌎 A completely free and open interplanetary microblogging platform 🚀
https://misskey-hub.net/
GNU Affero General Public License v3.0
10.09k stars 1.38k forks source link

fix(backend): URLをデコードしてからリクエストを送信するように #15015

Open AmaseCocoa opened 1 day ago

AmaseCocoa commented 1 day ago

What

Fix #15014

Why

現状はURLがデコードされていないのでリクエストに失敗してRSSが取得できないため

Additional info (optional)

Checklist

github-actions[bot] commented 1 day ago

このPRによるapi.jsonの差分 差分はありません。 Get diff files from Workflow Page

codecov[bot] commented 1 day ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 39.96%. Comparing base (a3d236c) to head (a0868b4). Report is 448 commits behind head on develop.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## develop #15015 +/- ## =========================================== - Coverage 41.74% 39.96% -1.79% =========================================== Files 1549 1563 +14 Lines 196555 197743 +1188 Branches 2767 3631 +864 =========================================== - Hits 82055 79027 -3028 - Misses 113939 118111 +4172 - Partials 561 605 +44 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


🚨 Try these New Features:

syuilo commented 1 day ago

APIの実装側はパラメータがエンコードされて送られてきているといった知識を持つべきではないから、HTTPについて関心を持つAPI実装を呼び出す側でハンドリングするのが良さそうです

AmaseCocoa commented 1 day ago

APIの実装側はパラメータがエンコードされて送られてきているといった知識を持つべきではないから、HTTPについて関心を持つAPI実装を呼び出す側でハンドリングするのが良さそうです

core/HttpRequestService.tsの方ですか? (HttpRequestService.send側で処理するようにするという認識をしています、違うかもしれませんが...)

AmaseCocoa commented 1 day ago

https://github.com/misskey-dev/misskey/pull/15015#issuecomment-2490258486 とりあえず↑の認識で進めました