본문으로 건너뛰기
cccskills
무료GitHub에 공개

api-controller-policy-attributes

Migrate API controller scope checks to framework-enforced authorization policies while preserving ownership checks.

한국어 요약은 준비 중입니다. 원문 설명을 표시하고 있습니다.

설치 방법 보기

포함된 파일(1)

  • SKILL.md2.3 KB

SKILL.md(원문)

설치하기 전에 에이전트에게 전달되는 지시 내용을 확인할 수 있습니다.

Context

Use this when an ASP.NET Core API controller currently calls VerifyUserHasAnyAcceptedScope(...) inside actions and the host already has hierarchical authorization policies registered.

Patterns

  • Keep the controller's class-level [Authorize] attribute so authentication still gates every action.
  • Define shared policy-name constants in src\JosephGuadagno.Broadcasting.Domain\Constants\AuthorizationPolicyNames.cs and consume them from policy registration, controller attributes, and reflection tests instead of repeating string literals.
  • Replace in-method scope checks with action-level [Authorize(Policy = ...)] attributes.
  • Use the cumulative role chain consistently:
    • RequireViewer for list/view/get/read actions
    • RequireContributor for add/create/update/modify actions
    • RequireAdministrator for delete actions
  • Leave ownership checks (GetOwnerOid(), IsSiteAdministrator(), per-resource owner comparisons) unchanged; policy migration should not broaden data visibility.
  • After migration, remove controller using Microsoft.Identity.Web.Resource; and any resource-specific scope references that are no longer needed.
  • Add a reflection-based test that asserts the expected policy per action so Phase 1 can verify the migration without rewriting direct controller behavior tests.

Examples

  • Shared constants: src\JosephGuadagno.Broadcasting.Domain\Constants\AuthorizationPolicyNames.cs
  • Controllers: src\JosephGuadagno.Broadcasting.Api\Controllers\EngagementsController.cs, SchedulesController.cs, SocialMediaPlatformsController.cs, UserPublisherSettingsController.cs, MessageTemplatesController.cs
  • Tests: src\JosephGuadagno.Broadcasting.Api.Tests\Controllers\ControllerAuthorizationPolicyTests.cs

Anti-Patterns

  • Hard-coding "RequireViewer" / "RequireContributor" / similar policy names in attributes, startup registration, or tests.
  • Removing class-level [Authorize] while adding policy attributes.
  • Mixing policy attributes with leftover VerifyUserHasAnyAcceptedScope(...) calls in the same controller.
  • Folding ownership logic changes into the policy migration phase.

리뷰

아직 리뷰가 없습니다. 사용해 본 소감을 남겨 주세요.

댓글

로그인 없이 쓸 수 있습니다. 사용 후기, 질문, 작동하지 않은 환경 등을 편하게 남겨 주세요.

아직 댓글이 없습니다. 첫 댓글을 남겨 보세요.

보내면 바로 공개됩니다. 이메일 주소나 전화번호 같은 개인 정보는 쓰지 마세요. URL은 2개까지입니다.

0 / 1,000자

이 양식은 reCAPTCHA로 보호되며 Google 개인정보처리방침 및 서비스 약관이 적용됩니다.

같은 저장소의 스킬

Standard collaboration patterns for all squad agents — worktree awareness, decisions, cross-agent communication

한국어 요약은 준비 중입니다. 원문 설명을 표시하고 있습니다.

jguadagno/jjgnet-broadcast182026년 8월 4일 업데이트

Shared hard rules enforced across all squad agents

한국어 요약은 준비 중입니다. 원문 설명을 표시하고 있습니다.

jguadagno/jjgnet-broadcast182026년 8월 4일 업데이트

Comprehensive AI prompt engineering safety review and improvement prompt. Analyzes prompts for safety, bias, security vulnerabilities, and effectiveness while providing detailed improvement recommendations with extensive frameworks, testing methodologies, and educational content.

한국어 요약은 준비 중입니다. 원문 설명을 표시하고 있습니다.

jguadagno/jjgnet-broadcast182026년 8월 4일 업데이트

Review API DTO implementations for contract/domain separation, mapping patterns, and REST compliance

한국어 요약은 준비 중입니다. 원문 설명을 표시하고 있습니다.

jguadagno/jjgnet-broadcast182026년 8월 4일 업데이트

Register shared claims transformation and hierarchical role policies in the API without breaking existing scope enforcement.

한국어 요약은 준비 중입니다. 원문 설명을 표시하고 있습니다.

jguadagno/jjgnet-broadcast182026년 8월 4일 업데이트

How to write comprehensive architectural proposals that drive alignment before code is written

한국어 요약은 준비 중입니다. 원문 설명을 표시하고 있습니다.

jguadagno/jjgnet-broadcast182026년 8월 4일 업데이트

jguadagno의 스킬 모두 보기

이 스킬의 문제 신고