Skip to content

Commit 8d2ee19

Browse files
author
github-actions
committed
chore: Java 스타일 수정
1 parent 4f55482 commit 8d2ee19

File tree

4 files changed

+25
-14
lines changed

4 files changed

+25
-14
lines changed

backend/src/main/java/io/f1/backend/domain/user/api/UserController.java

Lines changed: 10 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -5,8 +5,11 @@
55
import io.f1.backend.domain.user.app.UserService;
66
import io.f1.backend.domain.user.dto.SignupRequestDto;
77
import io.f1.backend.domain.user.dto.UserPrincipal;
8+
89
import jakarta.servlet.http.HttpSession;
10+
911
import lombok.RequiredArgsConstructor;
12+
1013
import org.springframework.http.ResponseEntity;
1114
import org.springframework.security.core.annotation.AuthenticationPrincipal;
1215
import org.springframework.stereotype.Controller;
@@ -24,18 +27,19 @@ public class UserController {
2427

2528
@DeleteMapping
2629
public ResponseEntity<Void> deleteCurrentUser(
27-
@AuthenticationPrincipal UserPrincipal userPrincipal, HttpSession httpSession) {
30+
@AuthenticationPrincipal UserPrincipal userPrincipal, HttpSession httpSession) {
2831
userService.deleteUser(userPrincipal.getUserId());
2932
logout(httpSession);
3033
return ResponseEntity.noContent().build();
3134
}
3235

3336
@PutMapping
3437
public ResponseEntity<Void> updateNickname(
35-
@AuthenticationPrincipal UserPrincipal userPrincipal,
36-
@RequestBody SignupRequestDto signupRequest, HttpSession httpSession) {
37-
userService.updateNickname(userPrincipal.getUserId(), signupRequest.nickname(),
38-
httpSession);
38+
@AuthenticationPrincipal UserPrincipal userPrincipal,
39+
@RequestBody SignupRequestDto signupRequest,
40+
HttpSession httpSession) {
41+
userService.updateNickname(
42+
userPrincipal.getUserId(), signupRequest.nickname(), httpSession);
3943
return ResponseEntity.noContent().build();
4044
}
41-
}
45+
}

backend/src/main/java/io/f1/backend/domain/user/app/UserService.java

Lines changed: 12 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -10,8 +10,11 @@
1010
import io.f1.backend.domain.user.dto.SignupResponseDto;
1111
import io.f1.backend.domain.user.entity.User;
1212
import io.f1.backend.global.util.SecurityUtils;
13+
1314
import jakarta.servlet.http.HttpSession;
15+
1416
import lombok.RequiredArgsConstructor;
17+
1518
import org.springframework.stereotype.Service;
1619
import org.springframework.transaction.annotation.Transactional;
1720

@@ -38,7 +41,7 @@ public SignupResponseDto signup(HttpSession session, SignupRequestDto signupRequ
3841

3942
private AuthenticationUser extractSessionUser(HttpSession session) {
4043
AuthenticationUser authenticationUser =
41-
(AuthenticationUser) session.getAttribute(OAUTH_USER);
44+
(AuthenticationUser) session.getAttribute(OAUTH_USER);
4245
if (authenticationUser == null) {
4346
throw new RuntimeException("E401001: 로그인이 필요합니다.");
4447
}
@@ -67,9 +70,9 @@ public void validateNicknameDuplicate(String nickname) {
6770
@Transactional
6871
public User initNickname(Long userId, String nickname) {
6972
User user =
70-
userRepository
71-
.findById(userId)
72-
.orElseThrow(() -> new RuntimeException("E404001: 존재하지 않는 회원입니다."));
73+
userRepository
74+
.findById(userId)
75+
.orElseThrow(() -> new RuntimeException("E404001: 존재하지 않는 회원입니다."));
7376
user.updateNickname(nickname);
7477

7578
return userRepository.save(user);
@@ -82,8 +85,10 @@ private void updateSessionAfterSignup(HttpSession session, User user) {
8285

8386
@Transactional
8487
public void deleteUser(Long userId) {
85-
User user = userRepository.findById(userId)
86-
.orElseThrow(() -> new RuntimeException("E404001: 존재하지 않는 회원입니다."));
88+
User user =
89+
userRepository
90+
.findById(userId)
91+
.orElseThrow(() -> new RuntimeException("E404001: 존재하지 않는 회원입니다."));
8792
userRepository.delete(user);
8893
}
8994

@@ -96,4 +101,4 @@ public void updateNickname(Long userId, String newNickname, HttpSession session)
96101
session.setAttribute(USER, AuthenticationUser.from(user));
97102
SecurityUtils.setAuthentication(user);
98103
}
99-
}
104+
}

backend/src/main/java/io/f1/backend/global/config/SecurityConfig.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,8 @@ public SecurityFilterChain userFilterChain(HttpSecurity http) throws Exception {
4242
.permitAll()
4343
.requestMatchers("/ws/**")
4444
.authenticated()
45-
.requestMatchers("/user/me").hasRole("USER")
45+
.requestMatchers("/user/me")
46+
.hasRole("USER")
4647
.anyRequest()
4748
.authenticated())
4849
.formLogin(AbstractHttpConfigurer::disable)

backend/src/main/java/io/f1/backend/global/util/SecurityUtils.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44
import io.f1.backend.domain.user.entity.User;
55

66
import jakarta.servlet.http.HttpSession;
7+
78
import org.springframework.security.authentication.UsernamePasswordAuthenticationToken;
89
import org.springframework.security.core.Authentication;
910
import org.springframework.security.core.context.SecurityContextHolder;

0 commit comments

Comments
 (0)