Default timeouts are set to zero, instead of 30 seconds
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Anfängerfreundlichkeit
- 35/100
- Issue-Typ
- Bug
- Klarheit
- Klar beschrieben
- Aktivitätsstatus
- Veraltet
- Tech-Stack
- php
- Bereich
- testing-qa
Rechercherichtung
Beginne bei src/Codeception/Module/WebDriver.php ungefähr in Zeile 362 und prüfe, wie die Standardwerte für connection_timeout und request_timeout konvertiert werden. Vergleiche dieses Verhalten mit den dokumentierten Standardwerten von 30 Sekunden und der Nullprüfung in php-webdriver's lib/Remote/RemoteWebDriver.php ungefähr in Zeile 101. Erledigt ist die Änderung, wenn ausgelassene Timeout-Einstellungen nicht mehr zu null werden oder dazu führen, dass curl_exec() unbegrenzt wartet.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Hey Guys,
Here https://codeception.com/docs/modules/WebDriver Codeception claims, that default timeouts (connection_timeout and request_timeout) are 30 seconds. Because of the bug, I think they are set to 0 now, which makes curl_exec() later to wait indefinitely.
Check this code https://github.com/Codeception/module-webdriver/blob/master/src/Codeception/Module/WebDriver.php#L362
$this->connectionTimeoutInMs = $this->config['connection_timeout'] * 1000;
$this->requestTimeoutInMs = $this->config['request_timeout'] * 1000;
by default both values are null, but multipling null * 1000 results in 0, not null. Later here https://github.com/php-webdriver/php-webdriver/blob/main/lib/Remote/RemoteWebDriver.php#L101 it is strictly compared with null, but since the value is 0 it passes the condition and sets timeout to 0 which makes curl to wait indefinitely.
- Vorherrschende Sprache
- PHP
- Sterne
- 39
- Forks
- 25
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beitragsleitfaden
Für dieses Repository ist kein Beitragsleitfaden indexiert
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus Codeception/module-webdriver
-
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 35/100
Codeception/module-webdriver#151 · 1 Kommentar ·
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 25/100
Codeception/module-webdriver#135 ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 38/100
Codeception/module-webdriver#130 ·
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 35/100
Codeception/module-webdriver#128 ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 45/100
Codeception/module-webdriver#126 ·
Alle Issues in Codeception/module-webdriver
Ähnliche Issues
-
Solved site promotion gate fails on runner PHP patch drift (expects 8.2.33, runner installs 8.2.34) Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
Automattic/blocks-engine#2161 ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 70/100
Automattic/static-site-importer#1824 ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 70/100
-
0. Needs triage bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
-
0. Needs triage 35-feedback bug
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100