Skip to content

Conversation

@github-actions
Copy link
Contributor

✨ PR Description

Purpose: Fix client payload escaping in rule engine to properly handle nested JSON objects and prevent parsing errors

Main changes:

  • Replaced safeParse with doubleParse to correctly handle JSON stringified payloads
  • Updated setClientPayload to properly apply escaping when setting new values
  • Restructured initialization flow to handle playground mode and optional parameters correctly

CHANGELOG

🐛 Bug Fixes

  • Fix client payload escaping in rule engine to properly handle nested JSON objects and prevent parsing errors.

Generated by LinearB AI and added by gitStream.
AI-generated content may contain inaccuracies. Please verify before using. We'd love your feedback! 🚀

@github-actions github-actions bot added the auto-deploy when exists in PR, will auto make release and auto deploy to prod label Aug 20, 2025
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

Copy link
Collaborator

@yeelali14 yeelali14 left a comment

Choose a reason for hiding this comment

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

LGTM

@yeelali14 yeelali14 merged commit 29a3c26 into develop Aug 20, 2025
10 checks passed
@yeelali14 yeelali14 deleted the LINBEE-18713-bump-gitstream-core-to-2.1.193 branch August 20, 2025 12:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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.

3 participants