/shareのurlパラメータに渡した内容が完全にデコードされてしまうため、日本語を含むURLがMFMに正しく認識されない

[alpaca-honke]

💡 Summary

/shareに渡すパラメータのうち、urlに渡した値のURIエンコードが全てデコードされてしまうため、URLに日本語などが含まれていた場合それ以降がURLとして認識されないため、すべて正しくURLとして認識されるようにしていただきたい

🥰 Expected Behavior

ふたつのオプションが考えられます
①urlパラメータに渡された値のうち、日本語部分などのみデコードを行わないことで、MFMに認識されるようになる
②MFMがURLを認識する際に日本語などを含んでいても認識されるようになる

🤬 Actual Behavior

/shareに渡すurlパラメータに日本語などが含まれると、投稿する際にそれ以降がURLとして認識されない

📝 Steps to Reproduce

1. /shareに渡すurlパラメータに、1度だけURIエンコードをかけた、日本語を含むURL（例えば、 https://github.com/alpaca-honke/twishare-to-misskey/wiki/よくある質問 ）を渡す
2. 投稿フォームに日本語にデコードされたURLが入り、日本語部分以降がURLとして認識されない

📌 Environment

💻 Frontend

• Model and OS of the device(s):Redme Note 10 JE, MiUI Global 14.0.12 (Based on Android 13 TKQ1.220905.001)
• Browser: Kiwi Browser 112.0.5615.137
• Server URL: ①misskey.io ②misskey.noellabo.jp
• Misskey:
  ①13.12.2 ②13.13.2

🛰 Backend (for server admin)

• Installation Method or Hosting Service:
• Misskey: 13.x.x
• Node: 18.x.x
• PostgreSQL: 15.x.x
• Redis: 7.x.x
• OS and Architecture:

備考

当然ではありますが、二重エンコードすれば正常に動作することを確認しています。
当分は二重エンコードしてtextの末尾に含めることで（修正後も）対応可能かと考えてはいます。

[acid-chicken]

挙動としては正しそう（つまり二重エンコードすべきよう）に思う

[tamaina]

日本語部分がURLであるかどうかをMisskey側では判断できないし…

[acid-chicken]

URL パラメータって URLSearchParams そのもののことではなく url パラメーターの話か

misskey/packages/frontend/src/pages/share.vue

Line 48 in f5dfb64

const url = urlParams.get('url');

[acid-chicken]

misskey/packages/frontend/src/pages/share.vue

Line 63 in f5dfb64

if (url) noteText += `${url}`;

ここを <${url}> とすれば直る

[tamaina]

あ〜

[esperecyan]

< > で囲むようにしても、URLに > が含まれている場合はうまくいきません。
https://misskey.io/share?url=https%3A%2F%2Fexample.com%2F%3C%E6%97%A5%E6%9C%AC%E8%AA%9E%3E

URLパーサーにかけた結果を本文に入れるのが良いのではないかと思います。

new URL('https://example.com/<日本語>').href
// ※URLとして取り扱えない文字列に対しては例外が出るので対処が必要

→ https://example.com/%3C%E6%97%A5%E6%9C%AC%E8%AA%9E%3E

[Sayamame-beans]

< > で囲むようにしても、URLに > が含まれている場合はうまくいきません。

こちらの現象はDiscord等でも同様のようです。
現状、MFMで使われる]などについても同様なので、気にしなくて良いのではないでしょうか…?

[anatawa12]

MFM層でURLに>がそのまま含まれてる場合に対応しないのは問題ないと思いますが、?url=に関しては対応したほうが良さそうに思います。URL Standardのparserはvalidation errorは出しますがパースは続行され>がURLEncodeされるのが定義されているため