Skip to content

fix(field-access-report): reference policy default in denial reason for root fields without explicit rule#114

Merged
ddebrunner merged 2 commits intomainfrom
issue-113
Jul 15, 2025
Merged

fix(field-access-report): reference policy default in denial reason for root fields without explicit rule#114
ddebrunner merged 2 commits intomainfrom
issue-113

Conversation

@carloseberhardt
Copy link
Contributor

Fix: Reference Policy Default in Denial Reason for Root Fields Without Explicit Rule

Summary

  • Denied root fields now reference the policy default and its condition in the reason string when no explicit rule is found.
  • Fixes test failure for introspection control policies (issue #113).
  • Ensures compliance with expected report output and improves clarity for denied access cases.

Details

  • Updates the denial reason logic in the field access report generation:

    • When a root field is denied due to the absence of a rule, and a exists, the denial reason now explicitly references the policy default and its condition.
    • Example:
  • This change ensures that the field access report is more informative and matches the expectations of the test suite and users.


Testing

  • All unit tests now pass, including:

  • Verified that the generated report for introspection control policies now includes the correct denial reason referencing the policy default.

Closes


Portions of the Content may be generated with the assistance of CursorAI

…or root fields without explicit rule\n\n- Denied root fields now reference the policy default and its condition in the reason string when no explicit rule is found.\n- Fixes test failure for introspection control policies.\n- Ensures compliance with expected report output and improves clarity for denied access cases.
Copy link
Contributor

@ddebrunner ddebrunner left a comment

Choose a reason for hiding this comment

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

GTG

@ddebrunner ddebrunner merged commit 0b37be0 into main Jul 15, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Test Failure: Field Access Report - Introspection Control Policies

2 participants