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

mention_usersとuser_icon_urlsのリクエストが多すぎるのでは? #7003

Closed
JunichiIto opened this issue Oct 28, 2023 · 5 comments
Closed
Assignees
Labels

Comments

@JunichiIto
Copy link
Contributor

提出物ページ( https://bootcamp.fjord.jp/products/17756 等)を開いて、Networkタブを見ると、mention_usersとuser_icon_urlsへのリクエストが繰り返し発生していました。

しかも、レスポンスを見ると全Userのアカウント情報やアイコンURLを返しているように見えます。
304で返ってきているとはいえ、これはちょっとサーバーに対して負荷をかけすぎなのでは?と心配になりました

d1 d2

あと、全Userの情報を返しているのもちょっと富豪的すぎるのではないかと思ったのですが、メンション機能を実現するにはこういう方法しかないのでしょうか?

(こういった点は考慮した上であえてこういう設計にしてあるとか、キャッシュを活用しているので問題ない、等々ので理由であれば、クローズしてもらって大丈夫です🙏)

@komagata komagata removed their assignment Jan 31, 2024
@komagata
Copy link
Member

komagata commented Feb 19, 2024

📝 対応方法

  1. ユーザーアイコンをユーザーIDから推測可能なURLにする。( /users/komagata/iconとかクラウドストレージのURL/bootcamp/user-icon/komagataとか)
  2. user_icon_urlsを使わないようにする。

1は現在、Cloud Storageにアップしたファイル名を使ってるので、ファイル名を変えるなどが必要かも。

Copy link

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

@github-actions github-actions bot added the stale label Apr 29, 2024
@JunichiIto JunichiIto removed the stale label May 1, 2024
Copy link

github-actions bot commented Jul 1, 2024

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

@github-actions github-actions bot added the stale label Jul 1, 2024
@JunichiIto JunichiIto removed the stale label Jul 1, 2024
@komagata komagata moved this to レビュー中 in bootcamp Aug 21, 2024
Copy link

このissue|PRは60日間更新がないため7日後にcloseします。closeしたくない場合はstaleラベルを外してください。

@github-actions github-actions bot added the stale label Aug 31, 2024
Copy link

github-actions bot commented Sep 7, 2024

このissue|PRはstaleラベルを付けた後7日間更新がないためcloseしました。

@github-actions github-actions bot closed this as completed Sep 7, 2024
@github-project-automation github-project-automation bot moved this from レビュー中 to 完成 in bootcamp Sep 7, 2024
@komagata komagata removed the stale label Sep 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

No branches or pull requests

3 participants