feat: admin 유저 정보 조회 req dto 변경 - #217

[sjk4618]

🔥Pull requests

⛳️ 작업한 브랜치

• Resolved: feat: admin 유저 정보 조회 req dto 변경 #217

👷 작업한 내용

admin 유저 정보 조회 req dto 변경

🚨 참고 사항

[coderabbitai]

Warning

Rate limit exceeded

@sjk4618 has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 8 minutes and 16 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between b7ba6b7 and 8228f0f.

📒 Files selected for processing (1)

• src/main/java/com/permitseoul/permitserver/domain/admin/base/api/controller/AdminController.java

📝 Walkthrough

Walkthrough

관리자 API의 사용자 권한 조회 요청 소스를 요청 본문(@RequestBody)에서 쿼리 파라미터(@RequestParam("email"))로 변경하고, 쿠키 생성 유틸의 상수 RESERVED_MAX_AGE 값과 주석이 일부 수정되었습니다.

Changes

코호트 / 파일(들)	변경 요약
관리자 API 파라미터 소스 변경 src/main/java/com/permitseoul/permitserver/domain/admin/base/api/controller/AdminController.java	getUserAuthority 메서드 시그니처를 @RequestBody UserAuthorityGetRequest request → @RequestParam("email") String email로 변경. 내부 호출을 adminService.getUserAuthority(email)로 수정. URLDecoder import 추가(해당 변경 내 사용 미확인).
쿠키 상수/주석 조정 src/main/java/com/permitseoul/permitserver/domain/auth/core/jwt/CookieCreatorUtil.java	RESERVED_MAX_AGE 값을 10L * 60 → 7L * 60으로 변경. COOKIE_MAX_AGE 주석이 변경되었으나 수치값은 유지(기존 2L * 60).

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• fix: 테스트용 쿠키 설정 #30: CookieCreatorUtil의 쿠키 관련 상수/설정 변경과 관련된 PR입니다.
• feat: admin 권한 관련 api - #209 #210: AdminController의 getUserAuthority 메서드 변경과 직접적으로 겹치는 PR입니다.
• feat: eventEntity 필드 수정 - #215 #216: CookieCreatorUtil의 상수 수정이 포함된 PR으로, 쿠키 수명 설정 변경이 중복될 수 있습니다.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Out of Scope Changes check	⚠️ Warning	CookieCreatorUtil.java 파일의 변경사항(RESERVED_MAX_AGE 수정)은 issue #217의 admin 사용자 정보 조회 DTO 변경과 무관한 범위 외 변경으로 보입니다.	CookieCreatorUtil.java의 변경사항을 제거하거나 별도의 PR에서 처리하세요. 이번 PR은 AdminController의 DTO 변경만 포함해야 합니다.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	제목이 변경 사항의 주요 내용을 명확하게 설명합니다: admin 사용자 정보 조회 요청 DTO 변경.
Description check	✅ Passed	설명이 변경 사항과 관련이 있으며, admin 사용자 정보 조회 요청 DTO 변경에 대해 언급합니다.
Linked Issues check	✅ Passed	코드 변경이 링크된 issue #217의 요구사항을 충족합니다: getUserAuthority API의 요청 파라미터 소스를 JSON 본문에서 쿼리 파라미터로 변경했습니다.
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 1

🤖 Fix all issues with AI agents

In
`@src/main/java/com/permitseoul/permitserver/domain/admin/base/api/controller/AdminController.java`:
- Around line 46-48: The query-param change removed DTO validation in
AdminController, so restore equivalent validation: annotate the controller
(AdminController) with `@Validated` (from
org.springframework.validation.annotation) and annotate the method parameter
(the String email in the getUserAuthority endpoint) with the javax/jakarta
validation constraints such as `@Email` and `@NotBlank` (i.e., use
`@RequestParam`("email") `@Email` `@NotBlank` final String email) and add the
necessary imports; this ensures request validation is performed when
adminService.getUserAuthority(email) is called.