Codeception / Codeception/module-phalcon4

Cookie not compatible with symfony/browser-kit

Offen
#24 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
Vorherrschende Sprache
PHP
Sterne
3
Forks
8
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

In the old version of `symfony/browser-kit`, the Cookie's `__construct` is

```
public function __construct(string $name, ?string $value, string $expires = null, string $path = null, string $domain = '', bool $secure = false, bool $httponly = true, bool $encodedValue = false, string $samesite = null) { ... }
```

And in the Phalcon4, using cookie set with `expires` is only allow integer, so there will be `TypeError` when running the test.

```
[TypeError] Argument 3 passed to Symfony\Component\BrowserKit\Cookie::__construct() must be of the type string or null, int given, called in /root/.composer/vendor/codeception/module-phalcon4/src/Codeception/Lib/Connector/Phalcon4.php on line 156
```

Maybe convert the timestamp to UTC String for old symfony/browser-kit?

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Rechercherichtung

Der fehlschlagende Aufruf befindet sich in Codeception/Lib/Connector/Phalcon4.php in Zeile 156; untersuche, wie der Phalcon4-Connector den ganzzahligen Cookie-Ablaufzeitpunkt an Symfony\Component\BrowserKit\Cookie::__construct übergibt. Reproduziere den betroffenen Test mit einem ganzzahligen Ablaufzeitpunkt und überprüfe anschließend, dass der Wert von der älteren BrowserKit-Signatur akzeptiert wird, ohne die anderen Cookie-Argumente zu ändern.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
php, symfony
Bereich
backend, testing
Issue-Typ
Bug
Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.