Problem/Motivation

We need to integrate Authorize.net ’s Accept Hosted API into the existing commerce_authnet module to support PCI-compliant credit card payments through their hosted payment form.

The integration must

  • Use the Accept Hosted API via getHostedPaymentPageRequest.
  • Support credit card payments only.
  • Support Customer Profiles, including creation and reuse of saved cards.
  • Follow a clean and minimal integration strategy. Preferably via a hook_form_alter(), or via a custom checkout pane similarly to Commerce Stripe if the hook_form_alter() solution doesn't work out somehow.

Acceptance Criteria

  • A new payment gateway plugin is defined.
  • The Accept Hosted payment form is embedded as an iframe at checkout.
  • Only card payments are supported.
  • Customer and Payment Profiles are created and reused via createCustomerProfileRequest and related APIs.
  • No sensitive payment information touches our servers (fully PCI SAQ-A compliant).
  • Code is extensible, does not introduce unnecessary complexity, and avoids the 3452993 patch structure.
  • Basic test coverage and documentation provided.
CommentFileSizeAuthor
#15 3531056-accept-hosted-14.patch84.25 KBvmarchuk
#14 3531056-accept-hosted-13.patch62.71 KBvmarchuk
#13 3531056-accept-hosted-12.patch62.63 KBvmarchuk
#12 3531056-accept-hosted-11.patch54.61 KBvmarchuk
#11 3531056-accept-hosted-10.patch53.68 KBvmarchuk
#10 3531056-accept-hosted-9.patch53.65 KBimbatman
#8 3531056-accept-hosted-8.patch53.37 KBvmarchuk
#7 3531056-accept-hosted-7.patch53.76 KBvmarchuk
#6 3531056-accept-hosted-6.patch53.63 KBvmarchuk
#5 3531056-accept-hosted-5.patch53.6 KBvmarchuk
#4 3531056-accept-hosted-4.patch53.4 KBvmarchuk

Issue fork commerce_authnet-3531056

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

vmarchuk created an issue. See original summary.

vmarchuk’s picture

vmarchuk
Primary language English
Location Rivne, UA
commented
Related issues: +#3452993: Add "Accept Hosted" payment gateway plugin

vmarchuk opened merge request !32

vmarchuk’s picture

vmarchuk
Primary language English
Location Rivne, UA
commented
StatusFileSize
new 3531056-accept-hosted-4.patch53.4 KB
vmarchuk’s picture

vmarchuk
Primary language English
Location Rivne, UA
commented
StatusFileSize
new 3531056-accept-hosted-5.patch53.6 KB
vmarchuk’s picture

vmarchuk
Primary language English
Location Rivne, UA
commented
StatusFileSize
new 3531056-accept-hosted-6.patch53.63 KB
vmarchuk’s picture

vmarchuk
Primary language English
Location Rivne, UA
commented
StatusFileSize
new 3531056-accept-hosted-7.patch53.76 KB
vmarchuk’s picture

vmarchuk
Primary language English
Location Rivne, UA
commented
StatusFileSize
new 3531056-accept-hosted-8.patch53.37 KB

imbatman made their first commit to this issue’s fork.

imbatman’s picture

imbatman commented
StatusFileSize
new 3531056-accept-hosted-9.patch53.65 KB

New patch ensures anonymous checkouts only send the email (no customer ID) to Authorize.Net, preventing all guest profiles from being assigned to customer ID 0. Authenticated users still send both ID and email. Fixes CIM profile misattribution.

vmarchuk’s picture

vmarchuk
Primary language English
Location Rivne, UA
commented
StatusFileSize
new 3531056-accept-hosted-10.patch53.68 KB
vmarchuk’s picture

vmarchuk
Primary language English
Location Rivne, UA
commented
StatusFileSize
new 3531056-accept-hosted-11.patch54.61 KB
vmarchuk’s picture

vmarchuk
Primary language English
Location Rivne, UA
commented
StatusFileSize
new 3531056-accept-hosted-12.patch62.63 KB

Added integration with webhooks.

vmarchuk’s picture

vmarchuk
Primary language English
Location Rivne, UA
commented
StatusFileSize
new 3531056-accept-hosted-13.patch62.71 KB
vmarchuk’s picture

vmarchuk
Primary language English
Location Rivne, UA
commented
StatusFileSize
new 3531056-accept-hosted-14.patch84.25 KB
tkiehne’s picture

tkiehne commented
Related issues: +#3019539: Expand Accept.js to include Accept.js UI

Similar work in #3019539; not sure if quite the same but has the same objective

vmarchuk’s picture

vmarchuk
Primary language English
Location Rivne, UA
commented

@tkiehne I think it's something else. We're working on integrating https://developer.authorize.net/api/reference/features/accept-hosted.html.

morbus iff’s picture

morbus iff
they/them
commented

@vmarchuk, what's the difference between this one and #3452993?

tkiehne’s picture

tkiehne commented

@vmarchuk - yes, it appears they are two different approaches, however, from an initial appraisal, this feature and the AcceptJsUI feature will interact in various parts of the module code. I have been using the AcceptJsUI patch in production for three years and the issue that created it has been ongoing since 2018. I think it wise that the maintainers provide some guidance as to what features will end up being supported/integrated given the interactions.

That said, I am interested in seeing where this implementation goes as it might provide for my needs, and if it ends up merged into contrib will be more stable.

vmarchuk’s picture

vmarchuk
Primary language English
Location Rivne, UA
commented
Status: Active » Needs review

jsacksick made their first commit to this issue’s fork.

  • jsacksick committed f92cabbe on 8.x-1.x authored by vmarchuk 
    feat: #3531056 Implement Accept Hosted API

By: @marchuk.vitaliy
By: @...
jsacksick’s picture

jsacksick commented
Status: Needs review » Fixed

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

vmarchuk opened merge request !37

  • vmarchuk committed 71d08388 on 8.x-1.x 
    Resolve #3531056 "Remove webhooks logging"

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.