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パラメータにエンコードされた日本語を含めると、Misskeyに入力された時にデコードされてしまい、日本語部分がURLとして認識されない #12

Closed
alpaca-honke opened this issue Jun 12, 2023 · 7 comments
Labels
Status: Close: invalid This doesn't seem right

Comments

@alpaca-honke
Copy link
Owner

概要

タイトルの通りなので、URLを含む全ての文字列をtextパラメータに含める変更を加える

現在

タイトルの通り

期待

Misskeyに入力された時にもエンコードされた状態が維持される

@alpaca-honke alpaca-honke added the Type: Problem: bug Something isn't working label Jun 12, 2023
@alpaca-honke
Copy link
Owner Author

Misskey側の問題でもあるので、misskey-dev/misskeyにもissueを立てる

@alpaca-honke
Copy link
Owner Author

待ってこの問題はTwishareでは起きてないかも

@alpaca-honke alpaca-honke added 0: need to check このissueが不適切な可能性があるが、まだ確定していないのでinvaildは付けられない時につけるタグ Status: Close: invalid This doesn't seem right and removed 0: need to check このissueが不適切な可能性があるが、まだ確定していないのでinvaildは付けられない時につけるタグ labels Jun 12, 2023
@alpaca-honke
Copy link
Owner Author

・ブラウザのJSが取得するURLは、エンコード済みのものである
・Twishareはそれにもう一度パーセントエンコーディングを施している状態ににゃる
・すると、二重にエンコードされた状態とにゃる
・これをMisskeyクライアント側でデコードすると正しくエンコードされた状態ににゃる

上記の理由からTwishareでは修正は不要だと判断
ただ、この方面の情報に関しては一応目をつけておく必要はあると思う

https://mkkey.net/notes/9fw7i4vuyf

@alpaca-honke alpaca-honke closed this as not planned Won't fix, can't repro, duplicate, stale Jun 12, 2023
@alpaca-honke
Copy link
Owner Author

ただ、Misskey側の問題としては存在するので、後でmisskey-dev/misskeyにissue立てたい

@alpaca-honke alpaca-honke removed the Type: Problem: bug Something isn't working label Jun 12, 2023
@alpaca-honke
Copy link
Owner Author

まって、二重エンコードってダメじゃなかったっけ?

@alpaca-honke
Copy link
Owner Author

二重エンコードに関しては別のissue立てる

@alpaca-honke
Copy link
Owner Author

二重エンコードされてる状態で正常に動いてるから、それを直すかはMisskey側の対応によるので保留

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status: Close: invalid This doesn't seem right
Projects
None yet
Development

No branches or pull requests

1 participant