Skip to content

Conversation

@linearbci
Copy link
Collaborator

✨ PR Description

Purpose: Fix error handling in RuleParser to preserve accurate payload size information when handling large PR requests that exceed size limits.

Main changes:

  • Modified error detection logic to distinguish between client-side and backend 413 errors using message content
  • Preserved original error message with correct size calculation when error originates from client-side validation
  • Added comprehensive test suite covering size calculation, error flow simulation, and fix verification scenarios

CHANGELOG

🐛 Bug Fixes

  • Fixed incorrect payload size reporting in error messages for oversized PR requests

Generated by LinearB AI and added by gitStream.
AI-generated content may contain inaccuracies. Please verify before using.
💡 Tip: You can customize your AI Description using Guidelines Learn how

@linearbci linearbci added the auto-deploy when exists in PR, will auto make release and auto deploy to prod label Dec 15, 2025
@linearbci linearbci requested a review from MishaKav December 15, 2025 13:46
Copy link

@orca-security-us orca-security-us bot left a comment

Choose a reason for hiding this comment

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

Orca Security Scan Summary

Status Check Issues by priority
Passed Passed Infrastructure as Code high 0   medium 0   low 0   info 0 View in Orca
Passed Passed SAST high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Secrets high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Vulnerabilities high 0   medium 0   low 0   info 0 View in Orca

@nivSwisa1 nivSwisa1 merged commit 168fc94 into develop Dec 15, 2025
12 checks passed
@nivSwisa1 nivSwisa1 deleted the bump-gitstream-core-to-2.1.234 branch December 15, 2025 13:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1 min review auto-deploy when exists in PR, will auto make release and auto deploy to prod

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants