Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CSRF-Token-Error #4

Closed
dpf-dd opened this issue Jan 17, 2022 · 8 comments
Closed

CSRF-Token-Error #4

dpf-dd opened this issue Jan 17, 2022 · 8 comments

Comments

@dpf-dd
Copy link

dpf-dd commented Jan 17, 2022

Moin! Ich hatte den Thread im Slack verfolgt und dein kleines Tool heute auch mal ausprobiert.
Danke erstmal dafür, dass ist eine sehr sinnvolle Extension für yForm :)

Ich kriege es allerdings nicht zum Laufen:
Bitte das Formular erneut absenden, bzw. die Aktion erneut ausführen (Fehler: CSRF-Token ungültig).

Das bekomme ich IMMER wenn ich im Table-Manager ein Feld editieren möchte. Ich hab Cache gelöscht, hab mich neu eingeloggt, aber es ist immer das gleiche. :(

Redaxo ist aktuell mit 5.13.2 und Server läuft auf PHP 8.0.13... any ideas?

@eaCe
Copy link
Member

eaCe commented Jan 17, 2022

Danke schon mal für's testen, wenn auch ohne Erfolg :/
Ich hatte bisher keinerlei Probleme in dieser Art. Allerdings habe ich das ganze auch noch nicht mit PHP > 8 getestet.
Mit yForm 4 läuft es bei mir in der Testumgebung.
Kannst du mir vielleicht sagen welche Felder du nutzt?
Oder vielleicht mal eine Testtabelle mit nur einem Textfeld probieren?
Aktuell habe ich leider noch keine Idee wo das Problem entstehen könnte.

@dpf-dd
Copy link
Author

dpf-dd commented Jan 17, 2022

yformui

Also hab jetzt mal ne neue Tabelle angelegt und wollte ein Feld erstellen... Siehe Screenshot...
Interessant: Das Select-Auswahlfeld für die col-size wird im ADD/EDIT-Modus des Formulars angezeigt, aber wenn es die Fehlermeldung schmeißt, dann fehlt das Feld unten über dem Button. Ich vermute, da zündet der Einklink-EP nicht, aber das ist ggf. ein anderes "Problem"...

Hab den Testserver auch mal auf PHP 7.4 gestellt, keine Änderung. Gleicher Fehler. :(

EDIT: Jetzt hab ich die yForm 4.0.0 ausm Installer mal mit dem aktuellen Stand des GitHub-Repos ersetzt. Hat auch nix gebracht.

@eaCe
Copy link
Member

eaCe commented Jan 19, 2022

Ich konnte es reproduzieren.
Es scheint als würde der Wert von ‚_csrf_token‘ mit der Breite überschrieben.
Das kann man z.B. hier beobachten: https://github.com/yakamara/redaxo_yform/blob/d252e5cb0f031e5731c063fa818eccca03473b18/lib/yform.php#L354

Wenn man dort folgendes dumpt kann man das sehen:
dump($ValueObject->getName());
dump($ValueObject->getValue());

Warum das so ist weiß ich noch nicht. Das ist mir bei yForm 3 nicht aufgefallen, auch nicht wenn nach 3 die 4er installiert wurde.

Ich werde das am Wochenende genauer untersuchen.

@olien
Copy link
Member

olien commented Jan 19, 2022

Hab gerade den selben Fehler. nur zur Info.
YForm 4.0.0
PHP 7.4.27

@eaCe eaCe closed this as completed in 063c440 Jan 21, 2022
@eaCe
Copy link
Member

eaCe commented Jan 21, 2022

@olien @dpf-dd ich habe den Fehler gefunden. Gerne noch mal mit der aktuellen Version testen.
Möglicherweise auch ein fix für das andere Problem (#1)? Da ich es bisher nicht reproduzieren konnte, kann ich das leider nicht einschätzen.

@olien
Copy link
Member

olien commented Jan 21, 2022

Ein träümchen. es funktioniert. danke sehr!

@dpf-dd
Copy link
Author

dpf-dd commented Jan 21, 2022

Jo, super! Bei mir geht es jetzt auch. Danke für den schnellen Fix! :)

@dpf-dd
Copy link
Author

dpf-dd commented Jan 21, 2022

Warum kriegt der Oli ein Herzchen und ich nur nen Daumen? schnief ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants