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

[MODIFY] /user/main시에 로그인되지 않은 유저는 401이 아닌 정상 응답을 보내도록 변경 #471

Merged
merged 3 commits into from
Jan 10, 2025

Conversation

kseysh
Copy link
Member

@kseysh kseysh commented Jan 9, 2025

📝 PR Summary

/user/main시에 로그인되지 않은 유저는 401이 아닌 정상 응답을 보내도록 변경하였습니다.
Authorization 헤더가 없을 시 아래와 같이 응답이 오도록 하였습니다.

{
    "user": {
        "status": "UNAUTHENTICATED",
        "name": "",
        "profileImage": "",
        "generationList": []
    },
    "operation": {
        "attendanceScore": 0.0,
        "announcement": ""
    },
    "isAllConfirm": false
}

🌴 Works

  • /user/main시에 로그인되지 않은 유저는 401이 아닌 정상 응답을 보내도록 변경

🌱 Related Issue

closed #470

@kseysh kseysh added the 🔧 Modify 기능 수정 label Jan 9, 2025
@kseysh kseysh requested a review from rlarlgnszx January 9, 2025 08:42
@kseysh kseysh self-assigned this Jan 9, 2025
Copy link

height bot commented Jan 9, 2025

Link Height tasks by mentioning a task ID in the pull request title or commit messages, or description and comments with the keyword link (e.g. "Link T-123").

💡Tip: You can also use "Close T-X" to automatically close a task when the pull request is merged.

Copy link
Contributor

@rlarlgnszx rlarlgnszx left a comment

Choose a reason for hiding this comment

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

수고하셨습니다~

@kseysh kseysh merged commit 6a0f388 into dev Jan 10, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[MODIFY] 메인 뷰 조회 시 비로그인 유저도 UNAUTHENTICATED 응답이 오도록 변경
2 participants