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

일정 - 회원의 일정 할당 여부 #143

Closed
f1v3-dev opened this issue Oct 18, 2024 · 2 comments · Fixed by #144
Closed

일정 - 회원의 일정 할당 여부 #143

f1v3-dev opened this issue Oct 18, 2024 · 2 comments · Fixed by #144
Assignees
Labels
bug Something isn't working

Comments

@f1v3-dev
Copy link
Member

🚨 어떤 버그인가요?

모임 상세 페이지에서 회원의 일정 할당 여부(check) 메서드에서 오류가 발생함.

@f1v3-dev f1v3-dev added the bug Something isn't working label Oct 18, 2024
@f1v3-dev
Copy link
Member Author

f1v3-dev commented Oct 18, 2024

문제 원인

public class SecurityEndpointPaths {


    private SecurityEndpointPaths() {
        // 인스턴스 생성 방지
    }

    public static final String[] WHITE_LIST = {
            "/api/v1/auth/oauth/**",
            "/api/v1/auth/**",
            "/api/v1/meetings/*/info",
            "/api/v1/meetings/*/times/**",
            "/api/v1/meetings/*/participants",
            "/api/v1/meetings/*/schedules/guests/**",
            "/api/v1/meetings/*/schedules/*"
    };

    public static final String[] USER_LIST = {
            "/api/v1/categories",
            "/api/v1/members/**",
            "/api/v1/meetings",
            "/api/v1/meetings/*/schedules/members",
            "/api/v1/meetings/*/schedules",
            "/api/v1/meetings/*/schedules/check"
    };

    public static final String[] ADMIN_LIST = {

    };
}

화이트 리스트의 "/api/v1/meetings/*/schedules/*" 로 인해,
유저 리스트에 명시해둔 "/api/v1/meetings/*/schedules/check" 경로까지 JWT 필터를 거치지 않는 문제가 존재함.

@f1v3-dev
Copy link
Member Author

해결

ScheduleController URI에 중간 계층을 하나 더 만들기

  • 비회원 : /guests + @
  • 회원 : /members + @

ex.
기존: /api/v1/meetings/*/schedules/check
수정: /api/v1/meetings/*/schedules/members/check

f1v3-dev added a commit that referenced this issue Oct 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants