Skip to content

Commit fa89a02

Browse files
oxdjwwoxdjww
authored
refactor: login 리팩토링 (#283)
1 parent 4442bee commit fa89a02

File tree

12 files changed

+49
-14
lines changed

12 files changed

+49
-14
lines changed

‎api/src/main/java/com/sponus/sponusbe/auth/controller/AuthController.java

+3-1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
package com.sponus.sponusbe.auth.controller;
22

3+
import static com.sponus.sponusbe.global.enums.ApiPath.*;
4+
35
import org.springframework.web.bind.annotation.GetMapping;
46
import org.springframework.web.bind.annotation.RequestHeader;
57
import org.springframework.web.bind.annotation.RequestMapping;
@@ -14,7 +16,7 @@
1416

1517
@Slf4j
1618
@RequiredArgsConstructor
17-
@RequestMapping("/api/v1/auth")
19+
@RequestMapping(AUTH_URI)
1820
@RestController
1921
public class AuthController {
2022

‎api/src/main/java/com/sponus/sponusbe/domain/bookmark/controller/BookmarkController.java

+3-1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
package com.sponus.sponusbe.domain.bookmark.controller;
22

3+
import static com.sponus.sponusbe.global.enums.ApiPath.*;
4+
35
import java.util.Collections;
46
import java.util.List;
57

@@ -24,7 +26,7 @@
2426

2527
@RestController
2628
@RequiredArgsConstructor
27-
@RequestMapping("/api/v1/me/announcements")
29+
@RequestMapping(ANNOUNCEMENT_URI)
2830
public class BookmarkController {
2931

3032
private final BookmarkService bookmarkService;

‎api/src/main/java/com/sponus/sponusbe/domain/notification/controller/FirebaseTestController.java

+3-1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
package com.sponus.sponusbe.domain.notification.controller;
22

3+
import static com.sponus.sponusbe.global.enums.ApiPath.*;
4+
35
import java.io.IOException;
46

57
import org.springframework.web.bind.annotation.PostMapping;
@@ -17,7 +19,7 @@
1719

1820
@Slf4j
1921
@RequiredArgsConstructor
20-
@RequestMapping("api/v1/notification")
22+
@RequestMapping(NOTIFICATION_URI)
2123
@RestController
2224
public class FirebaseTestController {
2325

‎api/src/main/java/com/sponus/sponusbe/domain/organization/club/controller/ClubController.java

+3-1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
package com.sponus.sponusbe.domain.organization.club.controller;
22

3+
import static com.sponus.sponusbe.global.enums.ApiPath.*;
4+
35
import org.springframework.web.bind.annotation.GetMapping;
46
import org.springframework.web.bind.annotation.PatchMapping;
57
import org.springframework.web.bind.annotation.PathVariable;
@@ -16,7 +18,7 @@
1618
import lombok.RequiredArgsConstructor;
1719

1820
@RestController
19-
@RequestMapping("/api/v2/clubs")
21+
@RequestMapping(CLUB_URI)
2022
@RequiredArgsConstructor
2123
public class ClubController {
2224
private final ClubService clubService;

‎api/src/main/java/com/sponus/sponusbe/domain/organization/company/controller/CompanyController.java

+3-1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
package com.sponus.sponusbe.domain.organization.company.controller;
22

3+
import static com.sponus.sponusbe.global.enums.ApiPath.*;
4+
35
import org.springframework.web.bind.annotation.GetMapping;
46
import org.springframework.web.bind.annotation.PatchMapping;
57
import org.springframework.web.bind.annotation.PathVariable;
@@ -16,7 +18,7 @@
1618
import lombok.RequiredArgsConstructor;
1719

1820
@RestController
19-
@RequestMapping("/api/v2/companies")
21+
@RequestMapping(COMPANY_URI)
2022
@RequiredArgsConstructor
2123
public class CompanyController {
2224
private final CompanyService companyService;

‎api/src/main/java/com/sponus/sponusbe/domain/organization/controller/OrganizationController.java

+3-1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
package com.sponus.sponusbe.domain.organization.controller;
22

3+
import static com.sponus.sponusbe.global.enums.ApiPath.*;
4+
35
import java.util.List;
46

57
import org.springframework.web.bind.annotation.DeleteMapping;
@@ -28,7 +30,7 @@
2830
import lombok.RequiredArgsConstructor;
2931

3032
@RestController
31-
@RequestMapping("/api/v2/organizations")
33+
@RequestMapping(ORGANIZATION_URI)
3234
@RequiredArgsConstructor
3335
public class OrganizationController {
3436
private final OrganizationService organizationService;

‎api/src/main/java/com/sponus/sponusbe/domain/organizationLink/controller/OrganizationLinkController.java

+3-1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
package com.sponus.sponusbe.domain.organizationLink.controller;
22

3+
import static com.sponus.sponusbe.global.enums.ApiPath.*;
4+
35
import org.springframework.web.bind.annotation.DeleteMapping;
46
import org.springframework.web.bind.annotation.GetMapping;
57
import org.springframework.web.bind.annotation.PatchMapping;
@@ -22,7 +24,7 @@
2224
import lombok.RequiredArgsConstructor;
2325

2426
@RequiredArgsConstructor
25-
@RequestMapping("/api/v1/organization-links")
27+
@RequestMapping(ORGANIZATION_LINK_URI)
2628
@RestController
2729
public class OrganizationLinkController {
2830
private final OrganizationLinkService organizationLinkService;

‎api/src/main/java/com/sponus/sponusbe/domain/portfolio/controller/PortfolioController.java

+3-1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
package com.sponus.sponusbe.domain.portfolio.controller;
22

3+
import static com.sponus.sponusbe.global.enums.ApiPath.*;
4+
35
import java.util.List;
46

57
import org.springframework.http.MediaType;
@@ -31,7 +33,7 @@
3133

3234
@Slf4j
3335
@RequiredArgsConstructor
34-
@RequestMapping("/api/v2/portfolio")
36+
@RequestMapping(PORTFOLIO_URI)
3537
@RestController
3638
public class PortfolioController {
3739
private final PortfolioService portfolioService;

‎api/src/main/java/com/sponus/sponusbe/domain/propose/controller/ProposeController.java

+3-1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
package com.sponus.sponusbe.domain.propose.controller;
22

3+
import static com.sponus.sponusbe.global.enums.ApiPath.*;
4+
35
import java.io.IOException;
46
import java.util.List;
57

@@ -33,7 +35,7 @@
3335

3436
@Slf4j
3537
@RequiredArgsConstructor
36-
@RequestMapping("/api/v1/proposes")
38+
@RequestMapping(PROPOSE_URI)
3739
@RestController
3840
public class ProposeController {
3941

‎api/src/main/java/com/sponus/sponusbe/domain/s3/S3TestController.java

+3-1
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,7 @@
11
package com.sponus.sponusbe.domain.s3;
22

3+
import static com.sponus.sponusbe.global.enums.ApiPath.*;
4+
35
import java.util.List;
46

57
import org.springframework.web.bind.annotation.DeleteMapping;
@@ -19,7 +21,7 @@
1921
@Slf4j
2022
@RequiredArgsConstructor
2123
@RestController
22-
@RequestMapping("/api/v1/s3")
24+
@RequestMapping(S3_URI)
2325
public class S3TestController {
2426

2527
private final S3Service s3Service;

‎api/src/main/java/com/sponus/sponusbe/global/enums/ApiPath.java

+15
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,15 @@
1+
package com.sponus.sponusbe.global.enums;
2+
3+
public class ApiPath {
4+
private static final String prefix = "/api/v2";
5+
public static final String AUTH_URI = prefix + "/auth";
6+
public static final String ANNOUNCEMENT_URI = prefix + "/me/announcements";
7+
public static final String NOTIFICATION_URI = prefix + "/notification";
8+
public static final String COMPANY_URI = prefix + "/companies";
9+
public static final String CLUB_URI = prefix + "/clubs";
10+
public static final String ORGANIZATION_URI = prefix + "/organizations";
11+
public static final String ORGANIZATION_LINK_URI = prefix + "/organization-links";
12+
public static final String PORTFOLIO_URI = prefix + "/portfolio";
13+
public static final String PROPOSE_URI = prefix + "/proposes";
14+
public static final String S3_URI = prefix + "/s3";
15+
}

‎core/core-infra-security/src/main/java/com/sponus/coreinfrasecurity/config/SecurityConfig.java

+4-4
Original file line numberDiff line numberDiff line change
@@ -41,8 +41,8 @@ public class SecurityConfig {
4141
"/",
4242
"/payment",
4343
"/api/v2/organizations/join/**",
44-
"/api/v2/organizations/login/**",
45-
"/api/v2/organizations/email/**",
44+
"/api/v2/auth/login/**",
45+
"/api/v2/auth/email/**",
4646
"/api/v2/report/**",
4747
"/api/v2/s3/**",
4848
"/api/v2/payments/**",
@@ -96,7 +96,7 @@ public SecurityFilterChain filterChain(HttpSecurity http) throws Exception {
9696
CustomLoginFilter loginFilter = new CustomLoginFilter(
9797
authenticationManager(authenticationConfiguration), jwtUtil, redisUtil
9898
);
99-
loginFilter.setFilterProcessesUrl("/api/v2/organizations/login");
99+
loginFilter.setFilterProcessesUrl("/api/v2/auth/login");
100100

101101
http
102102
.addFilterAt(loginFilter, UsernamePasswordAuthenticationFilter.class);
@@ -122,7 +122,7 @@ public SecurityFilterChain filterChain(HttpSecurity http) throws Exception {
122122
// Logout Filter
123123
http
124124
.logout(logout -> logout
125-
.logoutUrl("/api/v2/organizations/logout")
125+
.logoutUrl("/api/v2/auth/logout")
126126
.addLogoutHandler(new CustomLogoutHandler(redisUtil, jwtUtil))
127127
.logoutSuccessHandler((request, response, authentication) ->
128128
HttpResponseUtil.setSuccessResponse(

0 commit comments

Comments
 (0)