Script error on password reset

[graphicz]

Has anyone experienced this?

When doing a password reset and following the link in the received email and entering the new password and pressing submit you get a general script error:

A General Script Error Occurred Error:

Buyer cannot be determined at this time

Press the Browser back button and try again or contact the site owner

See: https://graphiczdev.uk/scripterror.mp4

It happens on several sites, a custom one, Smart default and Swift default.

No server log error entries

Sellerdeck .err file just shows Buyer cannot be determined entries

If anyone wants to test I can make an account for you on a test site to try.

Thank you

[Goz]

I have a vanilla 18.0.6 site for wholesale customers.

There are 2 ways to change the password - via the email link as you've described and from the account page when you've logged in.

I have tested both and they both work fine.

What version of Perl is being used? I'm using 5.016003

[graphicz]

Hi Andy - Thank you

Perl Version

5.016003

But it is 18.1 - fortunately my hosting company (Krystal) have no plans to upgrade their perl yet!

[graphicz]

It works perfectly on v18.0.5 - https://www.webeg.uk/abcd/ - on the same hosting/server

So this seems to be an 18.1 issue!

[Goz]

Best report it to SD and hope it is fixed in 18.1.1 because that's the version I'm waiting for

[graphicz]

The error only occurs when not logged in so the workaround is when creating a new acount give them a password and tell them to login and change the password in 'My Account'.

[NormanRouxel]

There was a change in ACTINIC.pm between 18.0.6 and 18.1.0:

18.0.6:

	# If we are using the authorisation file or if this is a call back
	# then we do not have the buyer key from the cookie to be able to
	# decrypt the buyer therefore we should not be calling GetBuyer()
	#
	if (!defined $::Session ||
		$::Session->IsCallBack() ||
		$::Session->IsAuthorisationFile())
		{
		ReportError("Buyer cannot be determined at this time", GetPath());
		}

18.1.0:

	# If we are processing a request from somewhere other than from the
	# the buyer's browser then there will be no business cookie and so no
	# buyer key to be able to decrypt the buyer therefore we should not be calling GetBuyer()
	#
	if ('' eq $ACTINIC::B2B->Get('UserKey'))
		{
		ReportError("Buyer cannot be determined at this time", GetPath());
		}

Perhaps this was not fully tested.

[graphicz]

Awesome. Thank you Norman.

[NormanRouxel]

I have a set of WinMerge command files that compares all .pl, pm and .js files between the Original folders for various versions of SellerDeck.
Here's the one that I just used to compare 18.1.0 with 18.0.6.

set wmdir=C:\Program Files\WinMerge
if not exist "%wmdir%\WinMergeu.exe" set wmdir=C:\Program Files (x86)\WinMerge
"%wmdir%\winmergeu.exe" /s /f "*.pl *.pm *.js" "V18.1.0 VDGA Original" "V18.0.6 VBHA Original"

Result is a list of files flagged as whether they are the same or have differences. Click an entry and you get the detailed differences.

[graphicz]

Thank you Norman