-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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とURIが異なるエンティティの照会に失敗する問題を修正 #15107
base: develop
Are you sure you want to change the base?
Conversation
it started complaining about that `true &&` all of a sudden
thanks to CenTdemeern1 for the `import` incantation
CenTdemeern1 had told me, but I got it wrong ☹
このPRによるapi.jsonの差分 |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #15107 +/- ##
===========================================
+ Coverage 38.91% 39.86% +0.95%
===========================================
Files 1563 1563
Lines 197745 197758 +13
Branches 3330 3635 +305
===========================================
+ Hits 76960 78844 +1884
+ Misses 120174 118307 -1867
+ Partials 611 607 -4 ☔ View full report in Codecov by Sentry. |
特定のサービスのドメインをハードコードし続けていくのはキリがなさそうだしメンテナンスの負荷が高そうな懸念があるわね |
URLとURIを異なるものにしなければならないユースケースってどういうのかしら |
元Issue(www.threads.netとthreads.net) |
そのメンテコストを外にするためにpsl npm packageを使ってるってことではあると思う |
threads側で同じものにできない理由がある…? |
ユースケース的にはActivity Pub用のidとwebページ上のページを分けたいってのはありそう。その使い分けとしてurl uriを分けることが適切なのかは怪しい気もするけど。(threads.netがやってること) リダイレクト処理ができなくはないけどCDNによってはVary対応がなかったり(CF等)複雑になったりがあるから別URLにした需要は十分あるかなと |
pslも結局は様々なサービスのドメインを列挙してるだけに見えるからpslがメンテナンスされ続けていく必要があるしメモリの観点からもあまり好ましくなさそう |
This reverts commit b5b94d5.
What
Public Suffix Listで同一性を判定するように
Why
Fix #15039
Additional info (optional)
https://activitypub.software/TransFem-org/Sharkey/-/merge_requests/773
Checklist