forter/forter-magento2

forter-magento2's Issues

AbstractApi has an undefined property forterErrorLogger

Error thrown:

Undefined property: Forter\Forter\Model\AbstractApi::$forterErrorLogger in /var/www/magento/vendor/forter/magento2-module-forter/Model/AbstractApi.php

Cause:

forterErrorLogger is not defined, see following class and line item:

forter-magento2/Model/AbstractApi.php

Line 176 in 79632b4

$this->forterErrorLogger->info($json);

SameSite cookie issue

We got these in our browser console when we browse our website with forter script installed

Cookie “forterToken” does not have a proper “SameSite” attribute value. Soon, cookies without the “SameSite” attribute or with an invalid value will be treated as “Lax”. This means that the cookie will no longer be sent in third-party contexts. If your application depends on this cookie being available in such contexts, please add the “SameSite=None“ attribute to it. To know more about the “SameSite“ attribute, read https://developer.mozilla.org/docs/Web/HTTP/Headers/Set-Cookie/SameSite

Cookie “forterToken” has been rejected for invalid domain.

Error when placing an order in Magento backend

Hello,

When we placed an order in the backend, we have an error "undefined variable". See below.

I added a simple product, with an existing customer, billing address = shipping address, and offline payment method and freeshipping as shipping method.

Module Version = 2.0.24

Jeremy

SyntaxError: Unexpected token '>'

For any browser/visitors who don't have ES6 support then they would get this error

We have traced our logs and we found this happening with some certain user agent like Daum 4.1, PhantomJS

Reference https://stackoverflow.com/questions/44203852/grunt-karma-phantomjs-syntaxerror-unexpected-token
It doesn't create a problem but it should be avoided

console error

(index):9187 Uncaught DOMException: Failed to execute 'removeChild' on 'Node': The node to be removed is not a child of this node.

Where the error happened

(function () {
var eu = "22g4zl{5jf5fjtlv1forxgiurqw1qhw2vwdwxv";
var siteId = 'f7794b8d2888';
function t(t,e){for(var n=t.split(""),r=0;r<n.length;++r)n[r]=String.fromCharCode(n[r].charCodeAt(0)+e);return n.join("")}function e(e){return t(e,-).replace(/%SN%/g,siteId)}function n(){var t="no"+"op"+"fn",e="g"+"a",n="n"+"ame";return window[e]&&window[e][n]===t}function r(){return!(!navigator.brave||"function"!=typeof navigator.brave.isBrave)}function o(){return document.currentScript&&document.currentScript.src}function i(t){try{F.ex=t,n()&&F.ex.indexOf(V.uB)===-1&&(F.ex+=V.uB),r()&&F.ex.indexOf(V.uBr)===-1&&(F.ex+=V.uBr),o()&&F.ex.indexOf(V.nIL)===-1&&(F.ex+=V.nIL),window.ftr__snp_cwc||(F.ex+=V.s),C(F)}catch(e){}}function a(t,e){function n(o){try{o.blockedURI===t&&(e(),document.removeEventListener(r,n))}catch(i){document.removeEventListener(r,n)}}var r="securitypolicyviolation";document.addEventListener(r,n),setTimeout(function(){document.removeEventListener(r,n)},2601e3)}function c(t,e,n,r){var o=!1;t="https://"+t,a(t,function(){r(!0),o=!0});var i=document.createElement("script");i.onerror=function(){if(!o)try{r(!1),o=!0}catch(t){}},i.onload=n,i.type="text/javascript",i.id="ftr__script",i.async=!0,i.src=t;var c=document.getElementsByTagName("script")[0];c.parentNode.insertBefore(i,c)}function u(t,e,n){var r=new XMLHttpRequest;if(a(t,function(){n(new Error("CSP Violation"),!0),isErrorEventCalled=!0}),"withCredentials"in r)r.open("GET",t,!0);else{if("undefined"==typeof XDomainRequest)return;r=new XDomainRequest,r.open("GET",t)}r.onload=function(){"function"==typeof e&&e(r)},r.onerror=function(t){if("function"==typeof n&&!isErrorEventCalled)try{n(t,!1),isErrorEventCalled=!0}catch(e){}},r.onprogress=function(){},r.ontimeout=function(){"function"==typeof n&&n("tim"+"eo"+"ut",!1)},setTimeout(function(){r.send()},0)}function d(){u(y,function(n){try{var r=n.getAllResponseHeaders().toLowerCase();if(r.indexOf(x.toLowerCase())<0)return;var o=n.getResponseHeader(x),i=t(o,--1);if(i){var a=i.split(":");if(a&&3===a.length){var c=a[0],u=a[1],d=a[2];switch(u){case"none":A=c+T;break;case"res":A=c+T+d;break;case"enc":A=c+e("1forxgiurqw1qhw2vq2(VQ(2vfulsw1mv");break;case"enc-res":for(var s="",v=0,l=0;v<20;++v)s+=v%3>0&&l<12?siteId.charAt(l++):F.id.charAt(v);var m=d.split(".");if(m.length>1){var h=m[0],w=m[1];A=c+T+h+"."+s+"."+w}}if(r.indexOf(S.toLowerCase())>=0){var p=n.getResponseHeader(S),g=t(p,--1);window.ftr__altd=g}j(V.dUAL),setTimeout(f,k,V.dUAL)}}}catch(y){}},function(t,e){i(e?V.uAS+V.cP:V.uAS)})}function f(t){try{var e=t===V.uDF?L:A;if(!e)return;var n=function(){try{Q(),i(t+V.uS)}catch(e){}},r=function(e){try{Q(),F.td=1new Date-F.ts,i(e?t+V.uF+V.cP:t+V.uF),t===V.uDF&&d()}catch(n){i(V.eUoe)}};c(e,void 0,n,r)}catch(o){i(t+V.eTlu)}}var s="fort",v="erTo",l="ken",m=s+v+l,h=10,w={write:function(t,e,n,r){void 0===r&&(r=!0);var o,i;if(n?(o=new Date,o.setTime(o.getTime()+24n6060*1e3),i="; expires="+o.toGMTString()):i="",!r)return void(document.cookie=escape(t)+"="+escape(e)+i+"; path=/");for(var a=1,c=document.domain.split("."),u=h,d=!0;d&&c.length>=a&&u>0;){var f=c.slice(-a).join(".");document.cookie=escape(t)+"="+escape(e)+i+"; path=/; domain="+f;var s=w.read(t);null!=s&&s==e||(f="."+f,document.cookie=escape(t)+"="+escape(e)+i+"; path=/; domain="+f),d=document.cookie.indexOf(t+"="+e)===-1,a++,u--}},read:function(t){var e=null;try{for(var n=escape(t)+"=",r=document.cookie.split(";"),o=32,i=0;i<r.length;i++){for(var a=r[i];a.charCodeAt(0)===o;)a=a.substring(1,a.length);0===a.indexOf(n)&&(e=unescape(a.substring(n.length,a.length)))}}finally{return e}}},p="13";p+="ck";var g=function(t){var e=function(){var e=document.createElement("link");return e.setAttribute("rel","pre"+"con"+"nect"),e.setAttribute("cros"+"sori"+"gin","anonymous"),e.onload=function(){document.head.removeChild(e)},e.onerror=function(t){document.head.removeChild(e)},e.setAttribute("href",t),document.head.appendChild(e),e};if(document.head){var n=e();setTimeout(function(){document.head.removeChild(n)},3e3)}},=3,y=e(eu||"22g4zl{5jf5fjtlv1forxgiurqw1qhw2vwdwxv"),T=t("1forxgiurqw1qhw2",-),x=t("[0Uhtxhvw0LG",-),S=t("[0Fruuhodwlrq0LG",-),A,L=e("(VQ(1fgq71iruwhu1frp2vq2(VQ(2vfulsw1mv"),E=e("(VQ(1fgq71iruwhu1frp2vqV2(VQ(2vfulsw1mv"),k=10;window.ftr__startScriptLoad=1*new Date;var U=function(t){var e=1e3,n="ft"+"r:tok"+"enR"+"eady";window.ftr__tt&&clearTimeout(window.ftr__tt),window.ftr__tt=setTimeout(function(){try{delete window.ftr__tt,t+="tt";var e=document.createEvent("Event");e.initEvent(n,!1,!1),e.detail=t,document.dispatchEvent(e)}catch(r){}},e)},C=function(t){var e=function(t){return t||""},n=e(t.id)+""+e(t.ts)+""+e(t.td)+""+e(t.ex)+""+e(p);w.write(m,n,1825,!0),U(n),window.ftr__gt=n},D=function(){var t=w.read(m)||"",e=t.split("_"),n=function(t){return e[t]||void 0};return{id:n(0),ts:n(1),td:n(2),ex:n(3),vr:n(4)}},q=function(){for(var t={},e="fgu",n=[],r=0;r<256;r++)n[r]=(r<16?"0":"")+r.toString(16);var o=function(t,e,r,o,i){var a=i?"-":"";return n[255&t]+n[t>>8&255]+n[t>>16&255]+n[t>>24&255]+a+n[255&e]+n[e>>8&255]+a+n[e>>16&15|64]+n[e>>24&255]+a+n[63&r|128]+n[r>>8&255]+a+n[r>>16&255]+n[r>>24&255]+n[255&o]+n[o>>8&255]+n[o>>16&255]+n[o>>24&255]},i=function(){if(window.Uint32Array&&window.crypto&&window.crypto.getRandomValues){var t=new window.Uint32Array(4);return window.crypto.getRandomValues(t),{d0:t[0],d1:t[1],d2:t[2],d3:t[3]}}return{d0:4294967296Math.random()>>>0,d1:4294967296Math.random()>>>0,d2:4294967296Math.random()>>>0,d3:4294967296Math.random()>>>0}},a=function(){var t="",e=function(t,e){for(var n="",r=t;r>0;--r)n+=e.charAt(1e3Math.random()%e.length);return n};return t+=e(2,"0123456789"),t+=e(1,"123456789"),t+=e(8,"0123456789")};return t.safeGenerateNoDash=function(){try{var t=i();return o(t.d0,t.d1,t.d2,t.d3,!1)}catch(n){try{return e+a()}catch(n){}}},t.isValidNumericalToken=function(t){return t&&t.toString().length<=11&&t.length>=9&&parseInt(t,10).toString().length<=11&&parseInt(t,10).toString().length>=9},t.isValidUUIDToken=function(t){return t&&32===t.toString().length&&/^[a-z0-9]+$/.test(t)},t.isValidFGUToken=function(t){return 0==t.indexOf(e)&&t.length>=12},t}(),V={uDF:"UDF",dUAL:"dUAL",uAS:"UAS",mLd:"1",eTlu:"2",eUoe:"3",uS:"4",uF:"9",tmos:["T5","T10","T15","T30","T60"],tmosSecs:[5,10,15,30,60],bIR:"43",uB:"u",uBr:"b",cP:"c",nIL:"i",s:"s"},b=function(t,e){for(var n=V.tmos,r=0;r<n.length;r++)if(t+n[r]===e)return!0;return!1};try{var F=D();try{F.id&&(q.isValidNumericalToken(F.id)||q.isValidUUIDToken(F.id)||q.isValidFGUToken(F.id))?window.ftr__ncd=!1:(F.id=q.safeGenerateNoDash(),window.ftr__ncd=!0),F.ts=window.ftr__startScriptLoad,C(F),window.ftr__snp_cwc=!!w.read(m),window.ftr__snp_cwc||(L=E);for(var I="for"+"ter"+".co"+"m",R="ht"+"tps://c"+"dn9."+I,B="ht"+"tps://"+F.id+"-"+siteId+".cd"+"n."+I,G="http"+"s://cd"+"n3."+I,O=[R,B,G],M=0;M<O.length;M++)g(O[M]);var N=new Array(V.tmosSecs.length),j=function(t){for(var e=0;e<V.tmosSecs.length;e++)N[e]=setTimeout(i,1e3V.tmosSecs[e],t+V.tmos[e])},Q=function(){for(var t=0;t<V.tmosSecs.length;t++)clearTimeout(N[t])};b(V.uDF,F.ex)?loadAlternate():(j(V.uDF),setTimeout(f,k,V.uDF))}catch(H){i(V.mLd)}}catch(H){}}
)();

Please send me a private message if you need some additional information

Avoid using unload

You can test with gtmetrix.com

The unload event does not fire reliably and listening for it can prevent browser optimizations like the Back-Forward Cache. Use pagehide or visibilitychange events instead.

Issue when change password in my account

Hello !
I use magento 2 version 2.4.1.
I have problem when change password in my account .
http://i.imgur.com/cJDgo7J.png
I checked and this error is caused wrong code in plugin module forter.
Path File: vendor/forter/magento2-module-forter/Plugin/Customer/Model/AccountManagement.php.
http://i.imgur.com/PiTf9kZ.png
Pls fix it.

I updated to avoid this error.You can refer to it.
http://i.imgur.com/X5mEL7Y.png
Pls confirm email [email protected] if module updated lastest.
Thanks!

getProductCategories always returns null if there is any catagory ids found

forter-magento2/Model/RequestBuilder/Cart.php

Line 146 in a7ee344

if ($categoryIds) {

should be corrected to

if (!$categoryIds) { return null; }

PHP requirement change between 2.4.11 and 2.4.12

Hi,

Is there any particular reason the minimum PHP version went up from 8.1 to 8.2 in version 2.4.12 of this extension? It means that users running M2 2.4.4 and 2.4.5 cannot update beyond version 2.4.11 of this package, since those versions only support PHP 8.1.

The commit that bumped the version doesn't include any explanation as to why: 0f4a8b3

Thanks.

Using wrong config path for declined email enabled configuration

forter-magento2/Model/Sendmail.php

Line 15 in a7ee344

const EMAIL_SERVICE_ENABLE = 'sendmail_on_decline/enabled';

the correct one should be sendmail_on_decline/sendmail_on_decline_enabled

as used in here

https://github.com/forter/forter-magento2/blob/a7ee344c2726525504c85b9db9c12549252f2cc1/Model/Config.php#L226C46-L226C93

Placing orders with Gift Cards not working

What is the issue

Forter decision throws the error: "Additional properties not allowed: beneficiaries"

What is the cause

On this line it add the beneficiary: https://github.com/forter/forter-magento2/blob/master/Model/RequestBuilder/Cart.php#L104

However a beneficiary can't be added according to the API documentation to a cart item:
https://portal.forter.com/app/developer/api/api/data-objects/Item