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

ユーザー画像のファイル名をユーザーIDに修正 #7747

Open
yocchan-git opened this issue May 1, 2024 · 2 comments
Open

ユーザー画像のファイル名をユーザーIDに修正 #7747

yocchan-git opened this issue May 1, 2024 · 2 comments
Assignees

Comments

@yocchan-git
Copy link
Contributor

概要

ユーザーアイコンの画像ファイル名が(user.id).pngだったが、下記のHotfixでそのロジックがなくなったため修正する。

  • セキュリティ的な観点(ファイルに個人情報などがあった場合)
  • ローカルでテストコードが落ちるため

(参考)

再現手順

  1. /users/:user_idにアクセス。
  2. ユーザーアイコンの上で「検証」をクリック。
  3. HTMLのimgタグのsrcを見て、元の画像名が表示されるのを確認
    Image from Gyazo

Image from Gyazo

期待される振る舞い

idにしてほしい
Image from Gyazo

https://github.com/fjordllc/bootcamp/blob/main/test/system/users_test.rb#L560
上記のテストが通るようにする

関連Issue

Ref: #7330

Copy link

github-actions bot commented Jul 7, 2024

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

@github-actions github-actions bot added the stale label Jul 7, 2024
@machida machida removed the stale label Jul 8, 2024
@komagata komagata moved this to やる in bootcamp Aug 21, 2024
Copy link

github-actions bot commented Sep 7, 2024

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

@github-actions github-actions bot added the stale label Sep 7, 2024
@machida machida removed the stale label Sep 11, 2024
@komagata komagata added the 2 label Oct 2, 2024
@komagata komagata added 3 and removed 2 labels Nov 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: 作業中
Development

No branches or pull requests

5 participants