Skip to content

fix(analytics): Fix for SQL Injection vulnerability in analytics endpoints#12312

Merged
inventvenkat merged 8 commits into
mainfrom
analytics_fix_merchant_jwt
May 15, 2026
Merged

fix(analytics): Fix for SQL Injection vulnerability in analytics endpoints#12312
inventvenkat merged 8 commits into
mainfrom
analytics_fix_merchant_jwt

Conversation

@su-shivanshmathur

@su-shivanshmathur su-shivanshmathur commented May 15, 2026

Copy link
Copy Markdown
Contributor

Type of Change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring
  • Dependency updates
  • Documentation
  • CI/CD

Description

Additional Changes

  • This PR modifies the API contract
  • This PR modifies the database schema
  • This PR modifies application configuration/environment variables

Motivation and Context

How did you test it?

Checklist

  • I formatted the code cargo +nightly fmt --all
  • I addressed lints thrown by cargo clippy
  • I reviewed the submitted code
  • I added unit tests for my changes where possible

@su-shivanshmathur su-shivanshmathur requested a review from a team as a code owner May 15, 2026 15:46
@semanticdiff-com

semanticdiff-com Bot commented May 15, 2026 •
edited
Loading

Copy link
Copy Markdown

Review changes with  SemanticDiff

Changed Files
File Status
  crates/analytics/src/query.rs  17% smaller

muditbhutani
muditbhutani previously approved these changes May 15, 2026
View reviewed changes
VenuMadhav2541
VenuMadhav2541 previously approved these changes May 15, 2026
View reviewed changes
@su-shivanshmathur su-shivanshmathur changed the title fix(analytics):Analytics Query Fix fix(analytics): analytics Query Fix May 15, 2026
@su-shivanshmathur su-shivanshmathur changed the title fix(analytics): analytics Query Fix fix(analytics): analytics query fix May 15, 2026
inventvenkat
inventvenkat requested changes May 15, 2026
View reviewed changes
Comment thread crates/analytics/src/query.rs
@inventvenkat inventvenkat enabled auto-merge May 15, 2026 17:35
@inventvenkat inventvenkat added this pull request to the merge queue May 15, 2026
Merged via the queue into main with commit f1a1068 May 15, 2026
42 of 46 checks passed
@inventvenkat inventvenkat deleted the analytics_fix_merchant_jwt branch May 15, 2026 20:00
This was referenced May 16, 2026
Merged
Merged
Merged
venkatakarthikm-maker pushed a commit that referenced this pull request May 18, 2026
@su-shivanshmathur @hyperswitch-bot
fix(analytics): analytics query fix (#12312)
cbddb94 
Co-authored-by: hyperswitch-bot[bot] <148525504+hyperswitch-bot[bot]@users.noreply.github.com>
@hrithikesh026 hrithikesh026 mentioned this pull request May 19, 2026
Merged
14 tasks
bernard-eugine pushed a commit that referenced this pull request May 19, 2026
@hrithikesh026 @su-shivanshmathur @hyperswitch-bot
fix(analytics): analytics query fix (#12312) (#12359)
e77333c 
Co-authored-by: Shivansh Mathur <104988143+su-shivanshmathur@users.noreply.github.com>
Co-authored-by: hyperswitch-bot[bot] <148525504+hyperswitch-bot[bot]@users.noreply.github.com>
@Shailesh-714 Shailesh-714 changed the title fix(analytics): analytics query fix fix(analytics): Fix for SQL Injection vulnerability in analytics endpoints May 20, 2026
AmitsinghTanwar007 pushed a commit that referenced this pull request May 22, 2026
@su-shivanshmathur @hyperswitch-bot @AmitsinghTanwar007
fix(analytics): analytics query fix (#12312)
7709ea2 
Co-authored-by: hyperswitch-bot[bot] <148525504+hyperswitch-bot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@inventvenkat inventvenkat inventvenkat approved these changes

@VenuMadhav2541 VenuMadhav2541 VenuMadhav2541 approved these changes

@muditbhutani muditbhutani muditbhutani left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@su-shivanshmathur @muditbhutani @inventvenkat @VenuMadhav2541