Authorize.Net Refactoring by StanislavSmetaninSSM · Pull Request #3 · dynamicweb/CheckoutHandlers.AuthorizeNetApi

StanislavSmetaninSSM · 2026-03-10T15:12:12Z

The task is to make the Authorize.Net version workable in the DW10.

What was did:

Refactoring: The codebase was fully refactored to align with the DW10 architecture.

Webhooks Integration: Added comprehensive webhook support. The provider interface now includes two new options:

Enable auto webhook registration: Webhooks are automatically registered during the first checkout transaction.
Force webhook re-registration: Resets existing webhook notifications associated with the current site URL and triggers a fresh registration. This setting automatically unchecks itself once the process is complete.

Security & Cleanup: Removed legacy settings for various checkout form templates. The integration now exclusively uses the Authorize.Net Hosted Payment Form, ensuring a more secure and standardized checkout process.

NOTE: The webhooks support is mandatory for this provider! It doesn't set the transaction number after redirect user back to DW site, so the only way to set it is a webhook notification!

The original task: https://dev.azure.com/dynamicwebsoftware/Dynamicweb/_workitems/edit/27013

…ponse.

…e opeation). Small code refactor and errors handling.

Copilot

Pull request overview

This PR is a comprehensive refactoring of the Authorize.Net checkout handler to align with the DW10 architecture. It replaces legacy model classes (in src/Model/, src/Enum/, src/API/) with properly structured models using DataContract/DataMember serialization, introduces a service layer architecture, and adds webhook support for transaction notifications.

Changes:

Replaced legacy plain model/enum/API classes with PascalCase-named models using DataContract attributes, organized under src/Models/, src/Helpers/, src/Constants/, src/Services/, and src/Exceptions/ directories.
Added comprehensive webhook management support including registration, update, deletion, and event type validation through AuthorizeNetService and related webhook models.
Introduced security helpers (HMAC validation, sensitive data masking), a structured logging system (AuthorizeNetLogger, AuthorizeNetRequestLogger), and organized constants for endpoints, response codes, and security settings.

Reviewed changes

Copilot reviewed 210 out of 212 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
src/Dynamicweb.Ecommerce.CheckoutHandlers.AuthorizeNetApi.csproj	Updated package versions, tags, copyright year, and added package icon
src/Services/AuthorizeNetService.cs	New core service handling Authorize.Net API interactions (payments, profiles, webhooks)
src/Services/AuthorizeNetHttpService.cs	New HTTP service wrapping HttpClient for API communication
src/Services/AuthorizeNetLogger.cs	New structured logger for Authorize.Net operations
src/Helpers/AuthorizeNetModelFactory.cs	New factory for creating billing/shipping address and line item models from orders
src/Helpers/*.cs	Various new helpers: security, HMAC, string manipulation, amount rounding, enum conversion
src/Models/*.cs	~60+ new model classes replacing legacy models with proper DataContract serialization
src/Constants/*.cs	New constants for endpoints, response codes, security settings, template folders, tags
src/Exceptions/AuthorizeNetApiException.cs	New custom exception for API errors
src/Model/*.cs (deleted)	Removed all legacy model classes
src/Enum/*.cs (deleted)	Removed all legacy enum classes
src/API/*.cs (deleted)	Removed all legacy API classes
src/Helper.cs (deleted)	Removed legacy monolithic helper class

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

… to copilot code review)

…of last name.

Copilot

Pull request overview

Copilot reviewed 210 out of 212 changed files in this pull request and generated 3 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot

Pull request overview

Copilot reviewed 209 out of 211 changed files in this pull request and generated 2 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot

Pull request overview

Copilot reviewed 209 out of 211 changed files in this pull request and generated 2 comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

1) Made NotificationEventType internal 2) Removed from SecuritySettings the MinSecureKeyLength - the AuthorizeNet will check it by itself and send an error if needed. 3) Added check for signatureKey in the ValidateSecurityConfiguration function

Copilot

Pull request overview

Copilot reviewed 209 out of 211 changed files in this pull request and generated no new comments.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

MatthiasSort · 2026-03-10T19:46:13Z

I've had some chats with AI.

Main areas to watch in QA:
• webhook/order correlation
• webhook retry behavior under slow or failing back-end processing
• fraud/held-for-review event handling
• auto-registration behavior on already-configured merchants
• saved card flows and refund/capture edge cases

StanislavSmetaninSSM added 11 commits February 13, 2026 01:44

Initial refactoring of Authorize.Net project.

1f20537

Refactored the code.

5cb81f4

Small code refactoring changes.

980f1e6

Fixed serialization error occurs when Authorize.Net returns Error res…

cbc01f5

…ponse.

Many changes: fixes for everything.

84c0c3c

Fixed Models, Fixed webhook notifications processing.

0872bee

Fixed according to Authorize.Net API - refund, capture, cancel.

598e32d

Simplified Capture code (since Authorize.Net supports only ONE captur…

98fb378

…e opeation). Small code refactor and errors handling.

Fixed errors handling.

d40a182

Final changes.

a45307c

Added a small explanation for refund operation.

5ba2a95

StanislavSmetaninSSM requested review from MatthiasSort, Copilot and dw-sha March 10, 2026 15:12

Copilot started reviewing on behalf of StanislavSmetaninSSM March 10, 2026 15:12 View session

Copilot AI reviewed Mar 10, 2026

View reviewed changes

Comment thread src/Helpers/AuthorizeNetModelFactory.cs Outdated

Comment thread src/Services/AuthorizeNetHttpService.cs Outdated

Comment thread src/Services/AuthorizeNetLogger.cs

Copilot AI mentioned this pull request Mar 10, 2026

[WIP] Address feedback on Authorize.Net refactoring regarding bug clarity #4

Closed

dynamicweb deleted a comment from Copilot AI Mar 10, 2026

StanislavSmetaninSSM added 2 commits March 11, 2026 02:02

Refactor Authorize.Net services to reuse shared HttpClient (according…

2f20454

… to copilot code review)

Fixed bug in the CreateBillAddress function - use First name instead …

1db1838

…of last name.

StanislavSmetaninSSM requested a review from Copilot March 10, 2026 16:12

Copilot started reviewing on behalf of StanislavSmetaninSSM March 10, 2026 16:13 View session

Copilot AI reviewed Mar 10, 2026

View reviewed changes

Comment thread src/Services/AuthorizeNetService.cs Outdated

Comment thread src/Models/Messages.cs Outdated

Comment thread src/Constants/PreparedMessages.cs Outdated

StanislavSmetaninSSM added 2 commits March 11, 2026 02:30

Added a check for the webhook ID before performing the delete operation.

6f3305f

Removed duplicates in the Models. Fixed typo

a8efa34

StanislavSmetaninSSM requested a review from Copilot March 10, 2026 16:42

Copilot started reviewing on behalf of StanislavSmetaninSSM March 10, 2026 16:42 View session

Copilot AI reviewed Mar 10, 2026

View reviewed changes

Comment thread src/Services/AuthorizeNetHttpService.cs Outdated

Comment thread src/Models/NotificationPayload.cs Outdated

Fixed according the code review.

970940d

StanislavSmetaninSSM requested a review from Copilot March 10, 2026 16:53

Copilot started reviewing on behalf of StanislavSmetaninSSM March 10, 2026 16:54 View session

Copilot AI reviewed Mar 10, 2026

View reviewed changes

Comment thread src/Helpers/SecurityHelper.cs

Comment thread src/Models/NotificationEventType.cs Outdated

Fixes:

4147f11

1) Made NotificationEventType internal 2) Removed from SecuritySettings the MinSecureKeyLength - the AuthorizeNet will check it by itself and send an error if needed. 3) Added check for signatureKey in the ValidateSecurityConfiguration function

StanislavSmetaninSSM requested a review from Copilot March 10, 2026 17:07

Copilot started reviewing on behalf of StanislavSmetaninSSM March 10, 2026 17:08 View session

Copilot AI reviewed Mar 10, 2026

View reviewed changes

MatthiasSort approved these changes Mar 10, 2026

View reviewed changes

StanislavSmetaninSSM merged commit 1dc0c1b into main Mar 11, 2026
4 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Authorize.Net Refactoring#3

Authorize.Net Refactoring#3
StanislavSmetaninSSM merged 17 commits into
mainfrom
ssm/27013-authorize_net_refactoring

StanislavSmetaninSSM commented Mar 10, 2026 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Uh oh!

MatthiasSort commented Mar 10, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

StanislavSmetaninSSM commented Mar 10, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

MatthiasSort commented Mar 10, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

StanislavSmetaninSSM commented Mar 10, 2026 •

edited

Loading