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

Misskeyのファイルをアップロードすると ファイルのドメインの一つ後ろに'/'が挿入される #13963

Open
1 task
mochi-sann opened this issue Jun 10, 2024 · 7 comments
Labels
⚠️bug? This might be a bug

Comments

@mochi-sann
Copy link

mochi-sann commented Jun 10, 2024

💡 Summary

misskeyで画像をアップロードしてファイルタブから画像のURLを確認するとファイルのURLの中でファイルのドメインの後ろに/が一つ追加されている

🥰 Expected Behavior

画像ファイルのURLのドメインの後ろに余計な/が入らない

🤬 Actual Behavior

画像ファイルのURLのドメインの後ろに/が入る
chromeではブラウザ側が修正するが wget コマンドなどで直接画像のURLを指定する場合など一部の環境では画像を取得できない

  • misskeyで表示されるファイルのURL : https://miiiiiiiii.mochi33.com//a832f4d0-157f-4457-a753-a6a9f8ee39ad.png
  • オブジェクトストレージ側のただしいURL https://miiiiiiiii.mochi33.com/%2Fa832f4d0-157f-4457-a753-a6a9f8ee39ad.png

📝 Steps to Reproduce

  1. misskeyのドライブに画像をアップロードする
  2. misskeyのドライブでファイルのを開いて下のURLを取得する
  3. ファイルのURLのドメインの後ろに/が一つ余計に追加されている

💻 Frontend Environment

* Model and OS of the device(s): Windows 11 home 23H2
* Browser: Google Chrome 125.0.6422.77
* Server URL: https://misskey.mochi33.com
* Misskey: 2024.5.0

🛰 Backend Environment (for server admin)

* Installation Method or Hosting Service: Docker
* Misskey:2024.5.0
* Node: 20.12.2
* PostgreSQL:  15.5
* Redis: 7.2.3
* OS and Architecture: Ubuntu
* Object Storage : CloudFlere R2

Do you want to address this bug yourself?

  • Yes, I will patch the bug myself and send a pull request
@mochi-sann mochi-sann added the ⚠️bug? This might be a bug label Jun 10, 2024
@mochi-sann
Copy link
Author

私が確認する限り misskey.systems でも同様のことが起きているようです

@t1nyb0x
Copy link
Collaborator

t1nyb0x commented Jun 11, 2024

自分のサーバー(misskey.t1nyb0x.cloud)で確認してみましたが、再現していなかったですね・・・

オブジェクトストレージの設定にあるBase URL末尾に / が入っているといったことはないのでしょうか?

@t1nyb0x
Copy link
Collaborator

t1nyb0x commented Jun 11, 2024

ファイルURLをよく見てみると、prefixが設定されていないように見えました。
自サーバーではprefixが設定されているため、 https://media.t1nyb0x.cloud/files/hogehoge.png となっています。

misskey.mochi33.comや、misskey.systemsではprefixにあたるものがなく、ドメイン直下にファイルがある状態になっているように見えました。
そのため、 https://{domain}/{prefix}/{filename} という構造でURLが決定されるはずが、
prefixがないために https://{domain}//{filename} となっていると予想したのですがどうなのでしょうか・・・

@mochi-sann
Copy link
Author

image
オブジェクトストレージを見てみたらprefixを設定してなかったです。多分そのような動きになっていると思います

@KisaragiEffective
Copy link
Collaborator

(prefixが空になっていたら意図しないURLになる可能性があることを警告したほうが良い…?)

@Sayamame-beans
Copy link
Collaborator

Sayamame-beans commented Jun 13, 2024

(空の時に2重にならないようにする方が妥当そうな気がします…)

@zyoshoka
Copy link
Contributor

Related to #7613 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚠️bug? This might be a bug
Projects
None yet
Development

No branches or pull requests

5 participants