Pull Request Explorer

Summary

Fix core: route UCS-only webhooks to UCS

Ensures UCS-only connectors always route webhook events to UCS, improving reliability and simplifying integration flow.

Health Assessment

• • PR resolved a routing bug with minimal changes and a single review, demonstrating efficient collaboration and low risk to production.

Summary

Add runtime S-test-ready label check for Cypress

Allows users to retry Cypress tests without pushing new commits, reducing unnecessary workflow runs.

Health Assessment

• • The PR had a moderate review cycle with several commits to refine the workflow logic, but overall the changes were straightforward and completed within a day.

Summary

Add optional deja dependency and feature flags

Introduces an optional instrumentation dependency that can be enabled for debugging without affecting default builds, ensuring configuration remains inert when the feature is off.

Health Assessment

• • The PR is a large, multi-file change split into a stack of smaller PRs; AI assisted writing; review required two rounds and a comment, but the feature is inert in default builds and fully tested for purity.

Summary

Fix checkout partial auth Cypress test

Fixes failing checkout partial authorization test, updating Nuvei test card to ensure reliable test execution.

Health Assessment

• • Quick fix with minimal changes, fast review and merge, indicating low complexity and high confidence.

Summary

Fix Novalnet dynamic fields email value mismatch

Fixes email value mismatch in Novalnet config for pmListDynamicFieldWithoutBilling and pmListDynamicFieldWithNames

Health Assessment

• • The PR was merged quickly with minimal review time, indicating a straightforward fix.

Summary

Add Stripe payment coverage for BankRedirect and PayLater

Expands automated test coverage for Stripe payment methods, ensuring reliability of iDEAL, OpenBankingUk, and PayLater flows. This reduces risk of runtime errors in production.

Health Assessment

• • Long cycle time and multiple merges indicate significant rework; extensive test changes suggest high complexity.

Summary

Gracefully handle unknown response variants

Improves the FIUU connector’s resilience by handling unknown enum variants and missing optional fields, reducing payment processing failures and enhancing reliability.

Health Assessment

• • The PR required multiple merges from main and a lengthy review cycle, indicating significant integration effort and potential coordination challenges.

Summary

Add Datatrans to UCS-only connectors

Routes the Datatrans connector through the Unified Connector Service by updating environment configs, ensuring consistent behavior across deployments.

Health Assessment

• • Quick review and merge, minimal changes, low risk.

Summary

Add customer ID to PayPal payment request

Enables passing customer identifiers for PayPal and card payments, improving tracking and personalization.

Health Assessment

• • Long cycle time and multiple rework commits suggest review bottlenecks; small scope but high iteration indicates possible complexity or testing issues.

Summary

Add connector credentials for Mifinity in CI

Adds Mifinity connector credentials to the Cypress test runner workflow, enabling automated end‑to‑end tests against the Mifinity integration.

Health Assessment

• • The PR was reviewed immediately and merged after a short cycle, indicating straightforward changes with minimal risk.

Summary

Add fields to 3ds exemption manager for migration

Enables migration from deprecated 3ds decision manager by adding business_country and metadata fields, improving compliance and flexibility.

Health Assessment

• • The PR added new fields to support migration, required multiple merges, but was reviewed quickly and merged after a 6‑day cycle.

Summary

Remove PayPal from mandatory CI connectors

Eliminates PayPal from mandatory connectors in Cypress tests, reducing intermittent failures and speeding up test runs.

Health Assessment

• • Quick change with minimal impact, fast review and merge.

Summary

Fix wrongly flagged XSS or SQLi attack for payment links

This PR resolves a security issue where payment links were incorrectly flagged as XSS or SQL injection attacks, ensuring legitimate links are processed correctly and improving user experience.

Health Assessment

• • The PR was merged within 3 hours with only one review round, indicating a straightforward fix. The large line count reflects substantial changes to the CSS generator, but the quick cycle time and minimal comments suggest low risk of regression.

Summary

Fix payment link XSS/SQLi flagging

Corrects false positives in payment link validation, improving security and user experience.

Health Assessment

• • Fast review and merge with minimal rework, indicating a straightforward bug fix with low complexity and risk.

Summary

Add Prop, PrivateLabel, and Dinacard card networks

Adds support for three new card networks, enabling accurate routing and classification of payments from BIN enrichment.

Health Assessment

• • PR added new card network variants with minimal code changes and quick review, indicating efficient implementation.

Summary

Introduce MCA domain type to reduce encryption calls

This change introduces a new MCA domain type that eliminates redundant encryption service calls, improving transaction processing speed and reducing operational costs.

Health Assessment

• • Fast cycle time and quick review indicate efficient collaboration, but the large code change and hotfix nature suggest moderate risk.

Summary

Fix payment link XSS/SQLi false positives

This PR replaces generic XSS validation for payment link CSS with a CSS generator that sanitizes unsafe selectors, allowing valid CSS while stripping malicious content, thus preventing false positives and improving user experience.

Health Assessment

• • The PR required several iterations to address review comments, indicating moderate complexity, but the overall cycle time remained within normal limits.

Summary

Remove MCA decryption call to improve routing performance

Reduces encryption service calls, improving routing latency and resource usage.

Health Assessment

• • Fast initial review but long cycle time before merge, suggesting scheduling or backlog delays rather than code complexity.

Summary

Address Rust 1.97.0 Clippy Lints

Fixes compiler lints to maintain code quality and compatibility with Rust 1.97, ensuring stable builds.

Health Assessment

• • Quick turnaround with a single review and minimal rework indicates an efficient process.

Summary

Map UCS connector errors in remaining payment gateway flows

Extends UCS connector error handling across all payment gateway flows, preserving detailed error information instead of generic 5xx responses to improve debugging and customer experience.

Health Assessment

• • Multiple merges and rework over a 5‑day cycle indicate complex integration and potential blockers; long review and cycle times suggest high effort and risk.

Summary

Template connector enum and placeholder error strings

This PR fixes hardcoded connector enum and Stripe error strings in the connector template test, ensuring generated connectors use correct enum variants and placeholder error messages.

Health Assessment

• • The PR addressed a template bug with minimal code changes, but the review process took over three days, indicating a slow review cycle yet low complexity and risk.

Summary

Add auth type field to UCS repeat payment

Adds an authentication type field to UCS repeat payment requests, enabling better control over payment flows and compliance.

Health Assessment

• • Long cycle time with multiple merge conflicts suggests integration friction, but the change is small and straightforward.

Summary

Add card info metadata fields

Adds optional metadata fields to the cards_info table and updates the API contract, enabling richer card data in responses and batch migrations.

Health Assessment

• • The PR required multiple iterations over 8 days, with many commits after the first review, indicating significant complexity and potential risk to the database schema and API contract.

Summary

Refactor Redsys connector for robust error handling

Improves resilience to API changes by adding unknown enum variants, optional fields, and better logging, reducing runtime errors and aiding debugging.

Health Assessment

• • The PR took over 3 days to merge with a single review round, indicating a moderate review cycle but no significant rework.

Summary

Fix Cypress tests to accept 422 status

Ensures currency conversion tests correctly handle 422 responses, improving test reliability and preventing false failures.

Health Assessment

• • The PR had a long review cycle with multiple merge commits, but the change was minimal and low risk.

Summary

Fix analytics null destination and exclude shadow events

Corrects connector event logs to handle null destinations and filter out internal shadow events, preventing 500 errors and ensuring accurate logs for merchants.

Health Assessment

• • The PR was merged within 12.5 hours, with a single review and two quick follow‑up commits, indicating a smooth, low‑risk bug fix with minimal scope.

Summary

Add resilience to Cashtocode response deserialization

Adds guards against empty bodies and non-JSON error payloads, ensuring graceful fallback. Enhances stability of payment sync and webhook handling for Cashtocode integration.

Health Assessment

• • Quick review and single commit indicate a straightforward fix with minimal risk.

Summary

Allow dashboard JWT access to legacy routes

Fixes a bug that blocked dashboard JWT authentication from accessing legacy customer and payment-method routes, restoring functionality for merchants using the modular payment-method service.

Health Assessment

• • Resolved quickly with a single commit and minimal code changes, indicating low complexity and risk.

Summary

Refactor Checkout connector error handling

Improves robustness of the Checkout integration by gracefully handling unknown API responses and missing fields, reducing failures and enhancing reliability for merchants.

Health Assessment

• • The PR was reviewed and merged within 11 hours, indicating a smooth and efficient review process. The moderate code changes and small file set suggest low complexity and minimal risk.

Summary

Add config to close idle DB connections

Users can now configure idle DB connection handling, preventing stale connections in long‑running pods and improving stability.

Health Assessment

• • PR merged after 3 days with moderate review time; moderate code churn; no AI involvement; configuration changes enhance reliability of database connections.