devol/wallabag
1
0
Fork 0
mirror of https://github.com/wallabag/wallabag.git synced 2024-12-14 09:25:51 +01:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #7265 from yguedidi/make-retries-an-instance-property-in-authenticatorsubscriber

Browse Source 
Make retries an instance property in AuthenticatorSubscriber
This commit is contained in:
Yassine Guedidi 2024-02-19 10:28:25 +01:00 committed by GitHub
commit 3c1a183e2d
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/Wallabag/CoreBundle/Guzzle/AuthenticatorSubscriber.php
View File

@ -18,7 +18,7 @@ class AuthenticatorSubscriber implements SubscriberInterface, LoggerAwareInterfa
// avoid loop when login failed which can just be a bad login/password
// after 2 attempts, we skip the login
public const MAX_RETRIES = 2;
private static $retries = 0;
private int $retries = 0;
/** @var SiteConfigBuilder */
private $configBuilder;
@ -97,9 +97,9 @@ class AuthenticatorSubscriber implements SubscriberInterface, LoggerAwareInterfa
$authenticator = $this->authenticatorFactory->buildFromSiteConfig($config);
$isLoginRequired = $authenticator->isLoginRequired($body);
$this->logger->debug('loginIfRequested> retry #' . self::$retries . ' with login ' . ($isLoginRequired ? '' : 'not ') . 'required');
$this->logger->debug('loginIfRequested> retry #' . $this->retries . ' with login ' . ($isLoginRequired ? '' : 'not ') . 'required');
if ($isLoginRequired && self::$retries < self::MAX_RETRIES) {
if ($isLoginRequired && $this->retries < self::MAX_RETRIES) {
$client = $event->getClient();
$emitter = $client->getEmitter();
@ -109,7 +109,7 @@ class AuthenticatorSubscriber implements SubscriberInterface, LoggerAwareInterfa
$event->retry();
++self::$retries;
++$this->retries;
}
}