<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
</head>
<body>
Etienne,<br>
<br>
Boy, do I hate it when this happens! :-(<br>
<br>
Hopefully those 2 bugs are squashed in 6.05-4 which I have just
released for you.<br>
<br>
Please can you do another quick update and see if it's actually
working now?<br>
<br>
Thanks!<br>
Jules.<br>
<br>
<div class="moz-cite-prefix">On 31/08/2020 18:19, Etienne Vogt
wrote:<br>
</div>
<blockquote type="cite"
cite="mid:WM!009b8d1190891b408dc06e1722b4ba521669f6b1191f81be667cf192d03c558b44deda0f9185ac649b5dfbe4a0d2a2ca!@mx.jul.es">On
Mon, 31 Aug 2020, Jules wrote:
<br>
<br>
<blockquote type="cite">Just release 6.05-3 including this fix.
<br>
</blockquote>
<br>
It seems there's also a missing argument on the call to
validUsername()
<br>
on line 264 (this time, it's already logged in users who get a
blank
<br>
page) :
<br>
<br>
[Mon Aug 31 19:08:12.540959 2020] [php7:error] [pid 1894] [client
145.238.184.196:33830] PHP Fatal error: Uncaught
ArgumentCountError: Too few arguments to function
NSSAuthenticator::validUsername(), 2 passed in
/opt/zendto/lib/NSSLDAPAuthenticator.php on line 264 and exactly 3
expected in /opt/zendto/lib/NSSAuthenticator.php:124\nStack
trace:\n#0 /opt/zendto/lib/NSSLDAPAuthenticator.php(264):
NSSAuthenticator->validUsername('vogt', Array)\n#1
/opt/zendto/lib/NSSMultiAuthenticator.php(130):
NSSLDAPAuthenticator->validUsername('vogt', Array, '')\n#2
/opt/zendto/lib/NSSDropbox.php(2221):
NSSMultiAuthenticator->validUsername('vogt', Array, '')\n#3
/opt/zendto/lib/NSSDropbox.php(635):
NSSDropbox->userFromCookie()\n#4 /opt/zendto/www/index.php(35):
NSSDropbox->__construct(Array)\n#5 {main}\n thrown in
/opt/zendto/lib/NSSAuthenticator.php on line 124
<br>
<br>
So here's another fix :
<br>
<br>
--- /opt/zendto/lib/NSSLDAPAuthenticator.php~ 2020-08-31
18:45:11.319698698 +0200
<br>
+++ /opt/zendto/lib/NSSLDAPAuthenticator.php 2020-08-31
19:10:59.681265140 +0200
<br>
@@ -261,7 +261,7 @@
<br>
<br>
// Chain to the super class for any further properties
to be added
<br>
// to the $response array:
<br>
- parent::validUsername($uname,$response);
<br>
+ parent::validUsername($uname,$response,$errormsg);
<br>
}
<br>
}
<br>
} else {
<br>
<br>
<br>
Also, the previous patch to authenticate() gives a PHP warning in
the logs :
<br>
<br>
[Mon Aug 31 19:11:04.883020 2020] [php7:warn] [pid 31231] [client
145.238.184.196:33866] PHP Warning: Declaration of
NSSMultiAuthenticator::authenticate(&$uname, $password,
&$response, $errormsg) should be compatible with
NSSAuthenticator::authenticate(&$uname, $password,
&$response, &$errormessage) in
/opt/zendto/lib/NSSMultiAuthenticator.php on line 0
<br>
<br>
</blockquote>
<br>
<pre class="moz-signature" cols="72">Jules
--
Julian Field MEng CEng CITP MBCS MIEEE MACM
The current UK shipping forecast:
East Dogger, Fisher: Variable 3 or less, increasing 4 at times. Slight,
occasionally smooth in east Fisher. Showers. Good.
<a class="moz-txt-link-abbreviated" href="http://www.Zend.To">www.Zend.To</a>
Twitter: @JulesFM
</pre>
</body>
</html>