Problem/Motivation

Passing the AVS info during a transaction should be made as a setting. It would be particularly useful for testing situations. Currently, the info is always passed - no matter what the mode is.

Proposed resolution

Create an option like, Pass AVS information Moneris payment gateway. Enabling that option will pass the information, other it won't.

Remaining tasks

N/A

User interface changes

A new option as checkbox will appear for the Moneris payment gateway add/edit form.

API changes

N/A

Data model changes

N/A

CommentFileSizeAuthor
#9 interdiff.txt1.31 KBsmccabe
#9 commerce_moneris-avs_option-2965010-9.patch2.75 KBsmccabe
#7 interdiff-3-7.txt896 bytessubhojit777
#7 2965010-7.patch2.24 KBsubhojit777
#3 2965010-3.patch2.32 KBsubhojit777

Comments

subhojit777 created an issue. See original summary.

subhojit777’s picture

subhojit777
he/him
Primary language Bengali
commented
subhojit777’s picture

subhojit777
he/him
Primary language Bengali
commented
Status: Active » Needs review
StatusFileSize
new 2965010-3.patch2.32 KB
travis-bradbury’s picture

travis-bradbury commented 
+    $form['should_pass_avs_info'] = [
+      '#type' => 'checkbox',
+      '#title' => $this->t('Pass AVS information'),
+      '#description' => $this->t('Enabling this option will pass the AVS information while making a transaction. It is recommended that it should not be enabled on production environment.'),
+      '#default_value' => isset($this->configuration['should_pass_avs_info']) ? $this->configuration['should_pass_avs_info'] : FALSE,
+    ];

Why is it recommended to disable it?

subhojit777’s picture

subhojit777
he/him
Primary language Bengali
commented

For security purpose.

travis-bradbury’s picture

travis-bradbury commented

That sounds backward. Address Verification Service is for fraud prevention. Moneris describes it as added security.

https://www3.moneris.com/mpgclassic/extras/efraud.php

Who would use it?

It is used by e-commerce and MOTO merchants that are looking for added security for their websites, as well as Merchant Resource Center transactions, to assist in limiting the number of fraudulent transactions.

If this module is going to make any recommendation it seems better to be pro-AVS than to suggest removing it.

subhojit777’s picture

subhojit777
he/him
Primary language Bengali
commented
StatusFileSize
new 2965010-7.patch2.24 KB
new interdiff-3-7.txt896 bytes

Got it.

travis-bradbury’s picture

travis-bradbury commented
Status: Needs review » Reviewed & tested by the community

Tests in sandbox and live worked for us.

smccabe’s picture

smccabe commented
Status: Reviewed & tested by the community » Needs review
StatusFileSize
new commerce_moneris-avs_option-2965010-9.patch2.75 KB
new interdiff.txt1.31 KB

Updated the wording to be more pro-avs and made it be on by default

smccabe’s picture

smccabe commented

  • smccabe committed 1fc129e on 8.x-2.x authored by subhojit777 
    Issue #2965010 by subhojit777, smccabe, tbradbury: Make AVS a setting
smccabe’s picture

smccabe commented
Status: Needs review » Fixed

Status: Fixed » Closed (fixed)

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