Skip to content

Conversation

@cabbage16
Copy link
Member

@cabbage16 cabbage16 commented Dec 7, 2025

๐ŸŽซ ๊ด€๋ จ ์ด์Šˆ

#321


๐Ÿ“„ ๊ฐœ์š”

JVM๊ณผ Spring Boot, ๊ธฐํƒ€ ๋ผ์ด๋ธŒ๋Ÿฌ๋ฆฌ๋“ค์˜ ๋ฒ„์ „์„ ์—…๊ทธ๋ ˆ์ด๋“œํ•˜๊ณ  ๊ทธ์— ๋”ฐ๋ฅธ ์—๋Ÿฌ๋ฅผ ํ•ด๊ฒฐํ–ˆ์Šต๋‹ˆ๋‹ค.


๐Ÿ”จ ์ž‘์—… ๋‚ด์šฉ

Spring Boot 4.0 Migration Guide์— ๋”ฐ๋ผ 4.0์œผ๋กœ ๋งˆ์ด๊ทธ๋ ˆ์ด์…˜ ํ•˜๊ธฐ ์ „ ์šฐ์„  3.5.X ๋ฒ„์ „์œผ๋กœ ์—…๊ทธ๋ ˆ์ด๋“œ ํ•˜๋Š” PR์ž…๋‹ˆ๋‹ค.
3.5.8 ์—…๊ทธ๋ ˆ์ด๋“œ์— ์„ฑ๊ณตํ•˜๋ฉด ํ•ด๋‹น ๊ฐ€์ด๋“œ์— ๋”ฐ๋ผ Java 25, Spring Boot 4.0 ๊นŒ์ง€ ์—…๊ทธ๋ ˆ์ด๋“œ ํ•  ๊ณ„ํš์ž…๋‹ˆ๋‹ค.

  • Java 17 -> 21
  • Spring Boot 3.0.5 -> 3.5.8
  • NoResourceFoundException ํ•ธ๋“ค๋ง(Spring 6.1๋ถ€ํ„ฐ ๋”์ด์ƒ ์ง์ ‘ 404๋ฅผ ๋ฐ˜ํ™˜ํ•˜์ง€ ์•Š๊ณ  ํ•ด๋‹น ์—๋Ÿฌ๋ฅผ ๋ฐœ์ƒ์‹œํ‚ด)
  • ๊ธฐํƒ€ ๋ผ์ด๋ธŒ๋Ÿฌ๋ฆฌ ๋ฒ„์ „์—… ๋ฐ ์ง€์› ์ค‘๋‹จ๋œ ๋ฌธ๋ฒ• ์ˆ˜์ •

๐Ÿ ํ™•์ธ ์‚ฌํ•ญ

  • ํ…Œ์ŠคํŠธ๋ฅผ ์™„๋ฃŒํ–ˆ๋‚˜์š”?
  • API ๋ฌธ์„œ๋ฅผ ์ž‘์„ฑํ–ˆ๋‚˜์š”?
  • ์ฝ”๋“œ ์ปจ๋ฒค์…˜์„ ์ค€์ˆ˜ํ–ˆ๋‚˜์š”?
  • ๋ถˆํ•„์š”ํ•œ ๋กœ๊ทธ, ์ฃผ์„, import ๋“ฑ์„ ์‚ญ์ œํ–ˆ๋‚˜์š”?

๐Ÿ™‹๐Ÿป ๋ง๋ถ™์ผ ๋ง

@Bamdoliro/maru-backend ๊ฐ์ž ํ™˜๊ฒฝ์—์„œ๋„ ํ•œ๋ฒˆ์”ฉ ์‹คํ–‰์‹œ์ผœ๋ด์ฃผ์‹œ๊ณ  ๋ฌธ์ œ ๋ฐœ์ƒํ•˜๋ฉด comment ๋ถ€ํƒ๋“œ๋ฆฝ๋‹ˆ๋‹ค!

cabbage16 and others added 8 commits November 27, 2025 01:12
- Java 17 -> 21
- Spring Boot 3.0.5 -> 3.5.8
- Spring Dependency Management 1.1.0 -> 1.1.7
- Spring Cloud 2022.0.2 -> 2025.0.0
- ์ƒ์œ„๋ฒ„์ „์—์„œ๋Š” 404๋ฅผ ์ž๋™์œผ๋กœ ๋ฐ˜ํ™˜ํ•˜์ง€ ์•Š๊ณ  NoResourceFoundException์„ ๋ฐœ์ƒ์‹œํ‚ค๋ฏ€๋กœ ์ด์— ๋Œ€ํ•œ ํ•ธ๋“ค๋ง ์ถ”๊ฐ€
- ๋นˆ FileMetadata๋กœ ์ธํ•œ ์˜ค๋ฅ˜ ํ•ด๊ฒฐ
@cabbage16 cabbage16 self-assigned this Dec 7, 2025
@cabbage16 cabbage16 requested a review from a team as a code owner December 7, 2025 12:26
@cabbage16 cabbage16 linked an issue Dec 7, 2025 that may be closed by this pull request
@github-actions
Copy link

github-actions bot commented Dec 7, 2025

์ด๋ ‡๊ฒŒ ๋ฆฌ๋ทฐํ•ด์ฃผ์„ธ์š”.

๐Ÿ–๏ธ ๊ทœ์น™

  1. ์™œ ๊ฐœ์„ ์ด ํ•„์š”ํ•œ์ง€ ์ถฉ๋ถ„ํžˆ ์„ค๋ช…ํ•ด์ฃผ์„ธ์š”.
  2. ๋‹ต์„ ์•Œ๋ ค์ฃผ๊ธฐ๋ณด๋‹ค๋Š” ์Šค์Šค๋กœ ๊ณ ๋ฏผํ•˜๊ณ  ๊ฐœ์„  ๋ฐฉ๋ฒ•์„ ์„ ํƒํ•  ์ˆ˜ ์žˆ๊ฒŒ ํ•ด์ฃผ์„ธ์š”.
  3. ์ฝ”๋“œ๋ฅผ ๊น”๋”ํ•˜๊ฒŒ ์œ ์ง€ํ•˜๊ณ , ์ผ๊ด€๋˜๊ฒŒ ๊ตฌํ˜„ํ•˜๋„๋ก ์•ˆ๋‚ดํ•ด ์ฃผ์„ธ์š”.
  4. ๋ฆฌ๋ทฐ ๊ณผ์ •์ด ์ˆ™์ œ๊ฒ€์‚ฌ๊ฐ€ ์•„๋‹Œ ํ•™์Šต๊ณผ์ •์œผ๋กœ ๋А๋‚„ ์ˆ˜ ์žˆ๊ฒŒ ๋ฆฌ๋ทฐํ•ด ์ฃผ์„ธ์š”.
  5. ๋ฆฌ๋ทฐ๋ฅผ ์œ„ํ•œ ๋ฆฌ๋ทฐ๋ฅผ ํ•˜์ง€ ๋งˆ์„ธ์š”. ํ”ผ๋“œ๋ฐฑ ํ•  ๊ฒŒ ์—†์œผ๋ฉด ์นญ์ฐฌํ•ด ์ฃผ์„ธ์š”.

๐Ÿฅฌ ์šฐ์„ ์ˆœ์œ„

๋ฆฌ๋ทฐ ์ค‘์š”๋„์— ๋”ฐ๋ผ ๋‹ค์Œ๊ณผ ๊ฐ™์€ prefix๋ฅผ ๋ถ™์—ฌ์ฃผ์„ธ์š”.

์˜ˆ์‹œ - P5) ์“ธ๋ชจ์—†๋Š” ์ฃผ์„ ์‚ญ์ œํ•ด์ฃผ์„ธ์š”.

  • P1: ๊ผญ ๋ฐ˜์˜ํ•ด์ฃผ์„ธ์š” (Request changes)
  • P2: ์ ๊ทน์ ์œผ๋กœ ๊ณ ๋ คํ•ด์ฃผ์„ธ์š” (Request changes)
  • P3: ์›ฌ๋งŒํ•˜๋ฉด ๋ฐ˜์˜ํ•ด ์ฃผ์„ธ์š” (Comment)
  • P4: ๋ฐ˜์˜ํ•ด๋„ ์ข‹๊ณ  ๋„˜์–ด๊ฐ€๋„ ์ข‹์Šต๋‹ˆ๋‹ค (Approve)
  • P5: ๊ทธ๋ƒฅ ์‚ฌ์†Œํ•œ ์˜๊ฒฌ์ž…๋‹ˆ๋‹ค (Approve)

@cabbage16 cabbage16 changed the title Build/#321 build: version up to java 21, spring boot 3.5 Dec 7, 2025
@cabbage16 cabbage16 changed the title build: version up to java 21, spring boot 3.5 build: migration to java 21, spring boot 3.5 Dec 7, 2025
@cabbage16
Copy link
Member Author

@MockBean deprecated ๐Ÿ‘€

Copy link
Member

@jyj1289 jyj1289 left a comment

Choose a reason for hiding this comment

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

@cabbage16 Sonar Cloud ํ„ฐ์ง€๋Š” ๊ฒƒ๋งŒ ํ™•์ธํ•ด์ฃผ์„ธ์š”

@cabbage16
Copy link
Member Author

cabbage16 commented Dec 8, 2025

@cabbage16 Sonar Cloud ํ„ฐ์ง€๋Š” ๊ฒƒ๋งŒ ํ™•์ธํ•ด์ฃผ์„ธ์š”

@jyj1289 CSRF disable ๋•Œ๋ฌธ์ธ๋ฐ ๊ธฐ์กด ์ฝ”๋“œ์—์„œ๋„ CSRF disable ํ•˜๊ณ ์žˆ์—ˆ์Šต๋‹ˆ๋‹ค. ์•„๋‹ˆ๋ฉด ์–ด์ฐจํ”ผ ์ง€๊ธˆ ๋งˆ๋ฃจ์˜ ์ธ์ฆ/์ธ๊ฐ€ ๋กœ์ง์—์„œ ์Šคํ”„๋ง ์‹œํ๋ฆฌํ‹ฐ๋ฅผ ์ „ํ˜€ ์‚ฌ์šฉํ•˜๊ณ ์žˆ๋Š” ๊ฒƒ ๊ฐ™์ง€ ์•Š์€๋ฐ ์•„์˜ˆ spring security ์˜์กด์„ฑ์„ ๋นผ๋Š”๊ฒƒ๋„ ๊ณ ๋ คํ•ด๋ณผ๋งŒ ํ•œ ๊ฒƒ ๊ฐ™์Šต๋‹ˆ๋‹ค

@cabbage16 cabbage16 requested a review from jyj1289 December 8, 2025 05:55
@sonarqubecloud
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
1 Security Hotspot

See analysis details on SonarQube Cloud

@cabbage16 cabbage16 merged commit 3bf7cdb into develop Dec 11, 2025
2 of 3 checks passed
@cabbage16 cabbage16 deleted the build/#321 branch December 11, 2025 15:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[๊ฐœ์„ ] ๋ฒ„์ „์—…

3 participants