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

fix(backend): original image is not original image on web page #59

Merged

Conversation

anatawa12
Copy link
Collaborator

@anatawa12 anatawa12 commented Sep 18, 2023

What

Why

misskey-dev#11760 (comment)

misskey-dev#11844

絶対にユーザの意図した動作ではないので再圧縮条件からサイズとEXIFを消す

Additional info (optional)

Checklist

  • Read the contribution guide
  • Test working in a local environment
  • (If needed) Add story of storybook
  • (If needed) Update CHANGELOG.md
  • (If possible) Add tests

@anatawa12 anatawa12 added the ⚠️Bug Something isn't working label Sep 18, 2023
@anatawa12
Copy link
Collaborator Author

@Sayamame-beans とりあえず8Kまでサイズがそのままになるようにした

@Sayamame-beans
Copy link

webpublicの上限を8Kにすると、別のサーバーにも8Kのまま渡っていきそうな気が少しします(そして恐らく、キャッシュするようになっていればそのサーバー上で別途保存されることになる)
連合で流れてきた画像を2Kにする処理がデフォルトで入っているなら良いのですが、そうでないなら、相手方のサーバーの容量を圧迫してしまう可能性が…?

@anatawa12
Copy link
Collaborator Author

その問題はそもそもMisskey側の問題なきがするし、リモートで8Kで見えるのがユーザの意図した動作だと思ってるんだけども違うのか

@Sayamame-beans
Copy link

Sayamame-beans commented Sep 19, 2023

いや、まあ…リモートからでも8Kで見えるのが理想ではあります。
ただ、 #60 で言及しているように、デフォルトが常に8Kまで膨れると困るという意見もあります。(とはいえ、webpublic用にwebp/pngとして可逆再圧縮は掛かっているのでしょうし、実際にどのくらい通信量/描画負荷に差が出るかというところではありますが)

@Sayamame-beans
Copy link

正直に言うと、今の私はどういう形に収まるのがベストなのか掴めずに居るので、言っていることが迷走しているかもしれません…

@anatawa12
Copy link
Collaborator Author

正直なところ通信量をそこまで圧縮すべきものだと思ってないのですよね...

@Sayamame-beans
Copy link

通信量

主に、上限があったり従量課金だったりするモバイル環境、あるいは低速な回線環境等ではツラいんじゃないかなとは思いますね
描画負荷はモバイル系で顕著かも?

@niwaniwa
Copy link
Collaborator

8kでもいい気はする...!
issueでも書きましたがconfigで設定できると嬉しいかも...?

@anatawa12
Copy link
Collaborator Author

とりあえず起動時configにします

@Sayamame-beans
Copy link

個人的には、バランスを考えると4Kがちょうどいいのかなと思い始めてきました。(元々が2Kしか無かったわけですし…)
他にも色々考えましたが、WQHDは2Kからの上がり幅としては少なく、8Kだと16倍でちょっと大きすぎるかな?という感じで、中間と考えると3Kかな?とも思いましたが、3Kは中途半端な値なので、そうするぐらいなら4Kで良いかな~と

@anatawa12
Copy link
Collaborator Author

とりあえずconfigにしました

@Sayamame-beans

This comment was marked as resolved.

@Sayamame-beans Sayamame-beans self-requested a review September 24, 2023 13:01
CHANGELOG.md Outdated Show resolved Hide resolved
packages/backend/src/core/DriveService.ts Show resolved Hide resolved
@anatawa12 anatawa12 merged commit f6e194e into niri-la:develop Sep 26, 2023
14 checks passed
@anatawa12 anatawa12 deleted the fix-original-image-is-not-original-image branch September 26, 2023 01:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚠️Bug Something isn't working packages/backend
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants