Skip to content

Conversation

@Baguette-bbang
Copy link
Collaborator

๐Ÿ”— ๊ด€๋ จ ์ด์Šˆ

#80 #72 #55 #64


๐Ÿ“Œ PR ์š”์•ฝ

qkrcodus and others added 30 commits July 24, 2025 12:42
- SwaggerConfig์— JWT SecurityRequirement ์ถ”๊ฐ€๋กœ API ๋ฌธ์„œ์— JWT ์ธ์ฆ ๋ช…์‹œ
- SecurityConfig์—์„œ frameOptions ์„ค์ •์„ Spring Security 6.1+ ๊ถŒ์žฅ ๋ฐฉ์‹(๋žŒ๋‹ค)์œผ๋กœ ๋ณ€๊ฒฝ : ๋…ธ๋ž€์ค„ ์ œ๊ฑฐํ–ˆ๋‹ค๋Š” ์˜๋ฏธ
- SecurityConfig์—์„œ /chatrooms/** ๊ฒฝ๋กœ ์ธ์ฆ ํ•„์š”ํ•˜๋„๋ก ์ถ”๊ฐ€
โ€ฆntication

[FEATURE] Chat ๋„๋ฉ”์ธ @AuthenticationPrincipal๋ฅผ ํ†ตํ•œ ์ธ์ฆ ์ ˆ์ฐจ ์ถ”๊ฐ€, ์Šค์›จ๊ฑฐ ๋ฌธ์„œ ์—…๋ฐ์ดํŠธ, ํ…Œ์ŠคํŠธ ํ† ํฐ ๋ฐ ์œ ์ €
  - ๋‹ค์ค‘ ์ด๋ฏธ์ง€ temp ์—…๋กœ๋“œ ์ง€์› (์ตœ๋Œ€ 10๊ฐœ)
  - users/{userId}/temp/ ๊ฒฝ๋กœ์— ์ž„์‹œ ์ €์žฅ
  - ํŒŒ์ผ ๊ฒ€์ฆ ๋ฐ ErrorCode ์ ์šฉ
  - 24์‹œ๊ฐ„ ๋งŒ๋ฃŒ ๊ธฐ์ค€ ๊ณ ์•„ ํŒŒ์ผ ํŒ๋ณ„ ๋กœ์ง
  - TempImageService ๊ธฐ๋Šฅ์„ ImageService๋กœ ํ†ตํ•ฉ
  - uploadTempImages() ๋ฉ”์„œ๋“œ๋กœ ์ผ๊ด„ ์ž„์‹œ ์—…๋กœ๋“œ ๊ธฐ๋Šฅ ์ถ”๊ฐ€
  - processContentAndMigrateImages()๋กœ ๋ฆฌ์น˜ํ…์ŠคํŠธ ์ฒ˜๋ฆฌ ๊ธฐ๋Šฅ ์ถ”๊ฐ€
  - ์ž„์‹œ ์ด๋ฏธ์ง€ URL ์ถ”์ถœ ๋ฐ ๋งˆ์ด๊ทธ๋ ˆ์ด์…˜ ๋กœ์ง ๊ตฌํ˜„
  - ์ •๊ทœ์‹ ํŒจํ„ด ์บ์‹ฑ์œผ๋กœ ์„ฑ๋Šฅ ์ตœ์ ํ™”
  - ๊ฒ€์ฆ ์ƒ์ˆ˜ ์ถ”๊ฐ€ (MAX_FILES_PER_UPLOAD, MAX_FILE_SIZE_BYTES)
  - ์ด๋ฏธ์ง€ ์ฒ˜๋ฆฌ์šฉ ๊ณตํ†ต ์œ ํ‹ธ๋ฆฌํ‹ฐ ๋ฉ”์„œ๋“œ ์ถ”๊ฐ€
  - Image ์—”ํ‹ฐํ‹ฐ์— updateS3Key() ๋ฉ”์„œ๋“œ ์ถ”๊ฐ€ํ•˜์—ฌ ๊ฒฝ๋กœ ์—…๋ฐ์ดํŠธ ๊ธฐ๋Šฅ ๊ตฌํ˜„
  - ImageRepository.findByS3Key() ๋ฐ˜ํ™˜ ํƒ€์ž…์„ Optional<Image>๋กœ ๋ณ€๊ฒฝ
  - TempImageService ์˜์กด์„ฑ ์ œ๊ฑฐ
  - ๋ชจ๋“  ์ž‘์—…์— ํ†ตํ•ฉ๋œ ImageService ์‚ฌ์šฉ
  - ๊ธฐ์กด API ์—”๋“œํฌ์ธํŠธ ํ˜ธํ™˜์„ฑ ์œ ์ง€
  - ์ž„์‹œ ์—…๋กœ๋“œ ์—”๋“œํฌ์ธํŠธ๋ฅผ ํ†ตํ•ฉ ์„œ๋น„์Šค๋กœ ์—…๋ฐ์ดํŠธ
  - moveFile() ๋ฉ”์„œ๋“œ๋กœ S3 ํŒŒ์ผ ์ด๋™ ๊ธฐ๋Šฅ ์ถ”๊ฐ€ (๋ณต์‚ฌ + ์‚ญ์ œ ํŒจํ„ด)
  - moveFiles()๋กœ ์ผ๊ด„ ํŒŒ์ผ ์ด๋™ ๊ธฐ๋Šฅ ์ถ”๊ฐ€
  - deleteFiles()๋กœ ์ผ๊ด„ ์‚ญ์ œ ๊ธฐ๋Šฅ ์ถ”๊ฐ€
  - getTempImageUrlPattern()์œผ๋กœ ๋™์  ์ •๊ทœ์‹ ํŒจํ„ด ์ƒ์„ฑ ๊ธฐ๋Šฅ ์ถ”๊ฐ€
  - verifyFileExists()๋กœ ํŒŒ์ผ ์กด์žฌ ํ™•์ธ ๊ธฐ๋Šฅ ์ถ”๊ฐ€
  - ํŒŒ์ผ ์กฐ์ž‘ ์‹œ ์—๋Ÿฌ ์ฒ˜๋ฆฌ ๋ฐ ๋กœ๊น… ๊ฐœ์„ 
- ์ด๋ฏธ์ง€ ๊ฒฝ๋กœ ๋ณ€ํ™˜ ๋กœ์ง ๊ตฌํ˜„ (temp -> permanent)
- ์ •๊ทœ์‹ ํŒจํ„ด ์ˆ˜์ •์œผ๋กœ temp URL ๋งค์นญ ๊ฐœ์„ 
- ํ…Œ์ŠคํŠธ์šฉ ์—”๋“œํฌ์ธํŠธ ๋ฐ ๋””๋ฒ„๊น… ๋กœ๊ทธ ์ถ”๊ฐ€
Feat/#64 ๋กœ๊ทธ๋ถ put, patch, delete ๊ตฌํ˜„ ๋ฐ ์ปจํŠธ๋กค๋Ÿฌ ์ˆ˜์ •
[FEATURE] ์ด๋ฏธ์ง€ ์ž„์‹œ ๊ฒฝ๋กœ ์—…๋กœ๋“œ
  - Image ์—”ํ‹ฐํ‹ฐ์— postId ์ปฌ๋Ÿผ ์ถ”๊ฐ€ํ•˜์—ฌ ๊ฒŒ์‹œ๊ธ€ ์—ฐ๊ฒฐ ๊ด€๊ณ„ ์„ค์ •
  - temp โ†’ permanent ์ด๋ฏธ์ง€ ๋ณ€ํ™˜ ์‹œ postId ์ž๋™ ์„ค์ • ๋กœ์ง ๊ตฌํ˜„
  - ๊ฒŒ์‹œ๊ธ€ ์ˆ˜์ • ์‹œ ์‚ญ์ œ๋œ ์ด๋ฏธ์ง€ ์ž๋™ ๊ฐ์ง€ ๋ฐ ์ •๋ฆฌ ๊ธฐ๋Šฅ ์ถ”๊ฐ€
  - ImageStorageService์— ๊ณตํ†ต URL ํŒจํ„ด ์œ ํ‹ธ๋ฆฌํ‹ฐ ๋ฉ”์„œ๋“œ ํ†ตํ•ฉ
Baguette-bbang and others added 9 commits July 31, 2025 00:53
  - ImageValidationService ๋ถ„๋ฆฌํ•˜์—ฌ ํŒŒ์ผ ๊ฒ€์ฆ ๋กœ์ง ์ „๋‹ด
  - ImagePathService ํ™•์žฅํ•˜์—ฌ ๊ฒฝ๋กœ ์ƒ์„ฑ + URL ํŒจํ„ด ์ฒ˜๋ฆฌ ํ†ตํ•ฉ
  - ImageStorageService ์ •๋ฆฌํ•˜์—ฌ ์ˆœ์ˆ˜ S3 ์ž‘์—…๋งŒ ๋‹ด๋‹น
  - ImageService ๋‹จ์ˆœํ™”ํ•˜์—ฌ ํ•ต์‹ฌ ๋น„์ฆˆ๋‹ˆ์Šค ๋กœ์ง์— ์ง‘์ค‘
  ImagePathService์— ๋ถ„์‚ฐ๋˜์–ด ์žˆ๋˜ ๊ฒ€์ฆ ๋กœ์ง๋“ค์„
  ImageValidationService๋กœ ์ด๋™ํ•˜์—ฌ ๋‹จ์ผ ์ฑ…์ž„ ์›์น™ ์ค€์ˆ˜
  ImageType์„ ์—”ํ‹ฐํ‹ฐ์—์„œ ์—ด๊ฑฐํ˜•์œผ๋กœ ์ ์ ˆํ•œ ์œ„์น˜๋กœ ์žฌ๋ฐฐ์น˜ํ•˜์—ฌ ๋„๋ฉ”์ธ ๊ตฌ์กฐ ๊ฐœ์„ 
- additionalPath๋ฅผ postId๋กœ ๋ณ€๊ฒฝํ•˜์—ฌ ํƒ€์ž… ์•ˆ์ •์„ฑ ํ–ฅ์ƒ
- ๋ถˆํ•„์š”ํ•œ String.valueOf() ํ˜ธ์ถœ ์ œ๊ฑฐ
- logBooks ํ•„๋“œ์— @Builder.Default ์–ด๋…ธํ…Œ์ด์…˜ ์ถ”๊ฐ€
- saveStatus ํ•„๋“œ์— @Builder.Default ์–ด๋…ธํ…Œ์ด์…˜ ์ถ”๊ฐ€
- @builder ๊ด€๋ จ ์ปดํŒŒ์ผ ๊ฒฝ๊ณ  ํ•ด๊ฒฐ
  - ๋งค์ผ ํ•œ๊ตญ์‹œ๊ฐ„ ์ƒˆ๋ฒฝ 3์‹œ์— ๊ณ ์•„ ์ด๋ฏธ์ง€ ์ •๋ฆฌ ์ž‘์—… ์ˆ˜ํ–‰
  - DivaryApplication์— @EnableScheduling ์–ด๋…ธํ…Œ์ด์…˜ ์ถ”๊ฐ€
  - ImageStorageService์— listTempFiles ๋ฉ”์„œ๋“œ ์ถ”๊ฐ€
  - .gitignore์— bin/ ํด๋” ์ œ์™ธ ์„ค์ • ์ถ”๊ฐ€
[FEATURE] ๊ฒŒ์‹œ๊ธ€ ์—…๋ฐ์ดํŠธ ์‹œ ์ด๋ฏธ์ง€ ์—…๋ฐ์ดํŠธ
@Baguette-bbang Baguette-bbang self-assigned this Jul 31, 2025
@Baguette-bbang Baguette-bbang added the ๐Ÿš€ deploy ์„œ๋ฒ„ ๋ฐฐํฌ label Jul 31, 2025
@Baguette-bbang Baguette-bbang requested review from qkrcodus, sereene and worhs02 and removed request for worhs02 July 31, 2025 14:43
Copy link
Collaborator

@sereene sereene left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

๊ณ ์ƒํ•˜์…จ์Šต๋‹ˆ๋‹ค!

@Baguette-bbang Baguette-bbang merged commit 6dfe0db into main Jul 31, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

๐Ÿš€ deploy ์„œ๋ฒ„ ๋ฐฐํฌ

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants