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

URLに日本語を含んでいた場合、正しくURLが解析されない。 #4430

Closed
Ouvill opened this issue Mar 6, 2019 · 5 comments
Closed
Assignees
Labels
💬Discussion Being discussed or needs discussion 🖍MFM The Misskey Flavored Markdown feature

Comments

@Ouvill
Copy link

Ouvill commented Mar 6, 2019

Summary

URLに日本語を含んでいた場合、正しくURLが解析されない。

以下のURLを投稿した時、日本語部分がURLとして認識されなかった。

https://ja.softether.org/1-features/1._極めて強力な_VPN_接続性

……そもそも日本語URLを使用しているサイトが滅べばいいのだけど

Expected Behavior

screenshot from 2019-03-07 00-18-11

日本語のURLも正しく認識されるべき。
投稿時に URL を以下のようにURLエンコードしておくと、正常にURLとして認識された。

https://ja.softether.org/1-features/1._%E6%A5%B5%E3%82%81%E3%81%A6%E5%BC%B7%E5%8A%9B%E3%81%AA_VPN_%E6%8E%A5%E7%B6%9A%E6%80%A7

投稿結果が上記画像。

Actual Behavior

screenshot from 2019-03-07 00-18-05

日本語部分がURLとして認識されていない。

Steps to Reproduce

  1. URLエンコードされていない日本語URL を含んだ投稿をする

Environment

  • OS: Ubuntu 18.10
  • Browser: Vivaldi 2.3.1440.60 (Stable channel) (64-bit)
  • Misskey 10.91.2
@Ouvill Ouvill added 🐛Bug Unexpected behavior packages/frontend Client side specific issue/PR labels Mar 6, 2019
@syuilo
Copy link
Member

syuilo commented Mar 7, 2019

URLに日本語は使えないので一応これで正しい挙動ですね

@syuilo syuilo added 💬Discussion Being discussed or needs discussion and removed 🐛Bug Unexpected behavior labels Mar 7, 2019
@Ouvill
Copy link
Author

Ouvill commented Mar 7, 2019

日本語を含んだ URL を使用しているサイトを呪うことにします。

@acid-chicken
Copy link
Member

Markdownなどのように<>で囲ったら日本語URLも許容するのはどうですか。

@tamaina
Copy link
Contributor

tamaina commented Mar 12, 2019

<…>も良いですが、MFMにはタグ構文があるのですこし好ましくない気がします(パーシング的には問題ない...?)。

[](https://ja.softether.org/1-features/1._極めて強力な_VPN_接続性)のように、リンク構文を用いることができるようにすればいいのではないでしょうか。「[]()()内は日本語テキストでもURLと認識するように」し、「[]内が空白でもURLこの構文であると認識して、この場合はURLを表示する」のようにする感じで。

@acid-chicken
Copy link
Member

acid-chicken commented Mar 12, 2019

/^<https?://.*?>/gのような判定にしてやるとおそらくパースの方は大丈夫だと思います。Markdownをいまだに名乗っている節もありますし、歩み寄れるところは歩み寄っていった方が良いでしょう。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
💬Discussion Being discussed or needs discussion 🖍MFM The Misskey Flavored Markdown feature
Projects
None yet
Development

No branches or pull requests

4 participants