-
-
Couldn't load subscription status.
- Fork 126
feat(policy): add settings to validate only input data for update #2152
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
Conversation
📝 WalkthroughWalkthroughA new validation option, Changes
Sequence Diagram(s)sequenceDiagram
participant Test as Regression Test
participant Prisma as Enhanced Prisma Client
participant Policy as Policy Layer
participant Validator as Schema Validator
Test->>Prisma: Create User with invalid email (no validation)
Test->>Prisma: Update User with inputOnlyValidationForUpdate=true
Prisma->>Policy: Check policy for update
Policy->>Validator: Validate input only (skips full entity validation)
Validator-->>Policy: Validation passes
Policy-->>Prisma: Allow update
Prisma-->>Test: Update succeeds
Test->>Prisma: Update User with default validation
Prisma->>Policy: Check policy for update
Policy->>Validator: Validate entire entity
Validator-->>Policy: Validation fails (invalid email)
Policy-->>Prisma: Reject update
Prisma-->>Test: Update fails
Assessment against linked issues
Assessment against linked issues: Out-of-scope changesNo out-of-scope changes were found. 📜 Recent review detailsConfiguration used: CodeRabbit UI 📒 Files selected for processing (3)
⏰ Context from checks skipped due to timeout of 90000ms (6)
🔇 Additional comments (4)
|
fixes #2025