Skip to content

Conversation

@github-actions
Copy link

@github-actions github-actions bot commented Jan 7, 2026

Automatic Release PR

This PR was automatically created after changes were pushed to develop.

Commits: 1 new commit(s)

Checklist

  • Review all changes
  • Verify CI passes
  • Approve and merge when ready for production

* fix: allow BankTxReturn refund without userData

The chargebackTx cronjob now handles two cases:
1. Entries WITH userData: check KYC/risk status as before
2. Entries WITHOUT userData: process without KYC/risk check

This enables automatic refunds for unassigned deposits that
have no associated user account.

* fix: require creditorData for automatic refund processing

Cronjob now only processes entries where chargebackCreditorData is set.
This prevents unnecessary errors from validateRequiredCreditorFields.

* fix: require creditorData for BuyCrypto bank refund processing

Cronjob now only processes bank refunds where chargebackCreditorData is set.
Checkout refunds don't need creditorData (handled by checkout provider).
)

* fix: exclude 4xx client errors from Application Insights failures

Add TelemetryProcessor to mark 4xx responses as success=true.
Only 5xx server errors should appear in the Failures dashboard.

This reduces noise from expected client errors (400 Bad Request,
401 Unauthorized, 404 Not Found) in failure metrics and alerts.

* fix: use correct type for responseCode (string, not number)

Application Insights SDK types define responseCode as string.
Use parseInt() for proper type conversion before comparison.
@github-actions
Copy link
Author

github-actions bot commented Jan 7, 2026

🤖 PR Review Bot

⚠️ Unverified Commits (1)

The following commits are not signed/verified:

  • 5b3cf3c [NO-TASK] Minor improvements (David May)
How to sign commits
# SSH signing (recommended)
git config --global gpg.format ssh
git config --global user.signingkey ~/.ssh/id_ed25519.pub
git config --global commit.gpgsign true

# Re-sign last commit
git commit --amend -S --no-edit
git push --force-with-lease

⚠️ Non-Conventional Commits (1)

The following commits don't follow conventional commit format:

  • 5b3cf3c [NO-TASK] Minor improvements

Expected: type(scope): description
Types: feat, fix, refactor, test, ci, docs, chore, perf, style, build, revert


⚠️ Security: 0 critical, 67 high vulnerabilities


This is an automated review. Please address the issues above.

@TaprootFreak TaprootFreak merged commit 230934e into master Jan 7, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants