Skip to content

feat(events): taking request and response both for detail extraction for kafka events#9993

Merged
Gnanasundari24 merged 2 commits intomainfrom
event_enrichment
Nov 4, 2025
Merged

feat(events): taking request and response both for detail extraction for kafka events#9993
Gnanasundari24 merged 2 commits intomainfrom
event_enrichment

Conversation

@su-shivanshmathur
Copy link
Contributor

@su-shivanshmathur su-shivanshmathur commented Oct 24, 2025 •
edited
Loading

Type of Change

This takes on fields from response fields of API and add it to the top level of log for hyperswitch API event.

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

Description

Addition on top of the PR #8780

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?

[enhancement]
"merchant_reference_id" = "txn_uuid"
"organization_name" = "part"
"organization_type" = "standard"
Screenshot 2025-10-27 at 8 12 05 PM Screenshot 2025-10-27 at 8 12 16 PM

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 October 24, 2025 19:01
@semanticdiff-com
Copy link

semanticdiff-com bot commented Oct 24, 2025 •
edited
Loading

Review changes with  SemanticDiff

Changed Files
File Status
  crates/router/src/services/api.rs  7% smaller

@codecov
Copy link

codecov bot commented Oct 24, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (main@0bc0c00). Learn more about missing BASE report.

Additional details and impacted files 
@@           Coverage Diff           @@
##             main    #9993   +/-   ##
=======================================
  Coverage        ?    3.94%           
=======================================
  Files           ?     1223           
  Lines           ?   299297           
  Branches        ?        0           
=======================================
  Hits            ?    11795           
  Misses          ?   287502           
  Partials        ?        0

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@su-shivanshmathur su-shivanshmathur changed the title taking request and response both for detail extraction for kafka events feat(events): taking request and response both for detail extraction for kafka events Oct 27, 2025
@su-shivanshmathur su-shivanshmathur self-assigned this Oct 27, 2025
@su-shivanshmathur su-shivanshmathur linked an issue Oct 27, 2025 that may be closed by this pull request
[FEATURE] Request/Response fields to be populated on top level #8804
Closed
2 tasks
@Gnanasundari24 Gnanasundari24 added this pull request to the merge queue Nov 4, 2025
Merged via the queue into main with commit e97c9c2 Nov 4, 2025
30 of 35 checks passed
@Gnanasundari24 Gnanasundari24 deleted the event_enrichment branch November 4, 2025 09:04
aadityaguptaa pushed a commit that referenced this pull request Nov 10, 2025
@su-shivanshmathur @hyperswitch-bot @aadityaguptaa
feat(events): taking request and response both for detail extraction …
07c4ad4 
…for kafka events (#9993)

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

@jarnura jarnura jarnura approved these changes

@hrithikesh026 hrithikesh026 hrithikesh026 approved these changes

Assignees

@su-shivanshmathur su-shivanshmathur

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[FEATURE] Request/Response fields to be populated on top level

4 participants

@su-shivanshmathur @jarnura @hrithikesh026 @Gnanasundari24