feat(events): Add payment reject audit events

[AnishKacham]

Type of Change

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

Description

Pass along request_state to payment_core
Modify the UpdateTracker trait to accept request state
Modify the PaymentReject implementation of UpdateTracker to generate an event

Additional Changes

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

Files changed

1. crates/router/src/core/payments/operations/payment_reject.rs
2. crates/router/src/events/audit_events.rs

Motivation and Context

Resolves #4669

How did you test it?

This requires FRM related flows to be tested, and may not be tested locally.

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

[semanticdiff-com]

Review changes with  

Changed Files

File	Status
crates/router/src/core/payments/operations/payment_reject.rs	0% smaller
crates/router/src/events/audit_events.rs	0% smaller

[tsdk02]

Hey @AnishKacham ,
Can you share the event json or a screenshot of the event,
Can refer to pr #4166 for reference.

For the UI in the screenshot, you can find the docs to set up Kafka, kafka-ui (the interface in the screenshot) and ClickHouse using Docker Compose here: https://github.com/juspay/hyperswitch/tree/main/crates/analytics/docs

and then try generating said events and monitor logs in kafka-ui.

Also the docs are self-explanatory but still if in any doubt ping us in this slack channel

[AnishKacham]

Hey @tsdk02 , I have everything setup for viewing the audit events. But I'm unsure of how to generate them. I've asked on the slack channel as well. Which endpoint do I hit from postman? I checked the hyper switch api reference buy did not find anything specific to "PaymentReject"

[tsdk02]

Hey @AnishKacham, I have confirmed with the concerned team, and PaymentReject is an internal operation and it cannot be tested independently.
Please make the above changes and your PR will be good to go.

[AnishKacham]

Hey @AnishKacham, I have confirmed with the concerned team, and PaymentReject is an internal operation and it cannot be tested independently. Please make the above changes and your PR will be good to go.

I'll make the changes

[tsdk02]

Hey @AnishKacham, a gentle reminder to add the changes to this PR, so it can be merged

[AnishKacham]

Hey @AnishKacham, a gentle reminder to add the changes to this PR, so it can be merged

Done. Let me know if something needs to be addressed.

[AnishKacham]

Rebased because of merge conflicts