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(backend): URLをデコードしてからリクエストを送信するように #15015

Open
wants to merge 9 commits into
base: develop
Choose a base branch
from

Conversation

AmaseCocoa
Copy link
Contributor

@AmaseCocoa AmaseCocoa commented Nov 21, 2024

What

Fix #15014

Why

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

Additional info (optional)

  • CHANGELOG.mdは(該当するPRが含まれたリリースがまだされていないので)変更していません (これが取り込まれる前にリリースされたら一応CHANGELOGも書いておきます やりました)
  • 実際にmi.amase.xyzでこのPull Requestを当てたMisskeyが動作しています

Checklist

  • Read the contribution guide
  • Test working in a local environment
  • (If needed) Add story of storybook
  • (If needed) Update CHANGELOG.md
  • (If possible) Add tests

@github-actions github-actions bot added the packages/backend Server side specific issue/PR label Nov 21, 2024
Copy link
Contributor

github-actions bot commented Nov 21, 2024

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

Copy link

codecov bot commented Nov 21, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 39.86%. Comparing base (a77ad7a) to head (69e662c).

Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #15015      +/-   ##
===========================================
- Coverage    39.96%   39.86%   -0.10%     
===========================================
  Files         1563     1563              
  Lines       197744   197746       +2     
  Branches      3631     3630       -1     
===========================================
- Hits         79027    78832     -195     
- Misses      118112   118307     +195     
- Partials       605      607       +2     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.


🚨 Try these New Features:

@syuilo
Copy link
Member

syuilo commented Nov 21, 2024

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

@AmaseCocoa
Copy link
Contributor Author

AmaseCocoa commented Nov 21, 2024

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

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

@AmaseCocoa
Copy link
Contributor Author

AmaseCocoa commented Nov 21, 2024

#15015 (comment)
とりあえず↑の認識で進めました

@kakkokari-gtyih kakkokari-gtyih added this to the v2024.11.1? milestone Nov 23, 2024
@syuilo
Copy link
Member

syuilo commented Nov 24, 2024

API関数を呼び出してる側という認識ですね👀
(具体的にどこでやるべきかはまだ見れてませんが)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
packages/backend Server side specific issue/PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

api/fetch-rssにエンコードされたURLを送るとエラーが発生する
3 participants