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

Aktuelle Dev-Version: Table Manager: CSRF-Token ungültig wenn Datensatz via Watson aufgerufen wird #1045

Closed
alxndr-w opened this issue May 17, 2021 · 6 comments

Comments

@alxndr-w
Copy link
Contributor

image

Aufruf über die GUI von Watson

@christophboecker
Copy link
Contributor

Den Effekt gibt es auch in anderen Zusammenhängen. Ich gebe z.B. einen "Löschen nicht möglich"-Fehler aus mit dem Hinweis, wo genau der Datensatz noch referenziert ist. Der Hinweis hat einen Link, der direkt das Formular des fehlerauslösenden Datensatzes aufruft. Klappt grundsätzlich gut, aber mit CSFR-Fehler wie von Alexander beschrieben. Ich nutze die Github-Version.

dergel added a commit that referenced this issue May 26, 2021
@dergel
Copy link
Member

dergel commented May 26, 2021

Die Debug XXX Meldung ist draussen.

Leider muss das ansonsten so sein. Man könnte zwar diskutieren, warum das auch bei edit ohne sent so sein muss, aber in der YForm passieren je nach Definition auch Dinge beim reinen Aufruf der Editiermaske. Deswegen ist der benötigte CSRF Key nötig.

Aber man kann die Links ja ergänzen, damit es richtig abläuft:

$_csrf_key = rex_yform_manager_table::get('tabellenname')->getCRSFKey()

In der URL mit der Edit/Delete Aufruf noch das ergänzen + rex_csrf_token::factory($_csrf_key)->getUrlParams())

Sofern ok so, gerne Issue schliessen

@christophboecker
Copy link
Contributor

Ah, cool! Was es nicht alles gibt!

@christophboecker
Copy link
Contributor

Müsste es nicht getCSRFKeyheißen und nicht getCRSFKey ?

@alxndr-w
Copy link
Contributor Author

Bestimmt hat @dergel gehofft, dass es nie einem auffällt. Und nun das.

https://github.com/yakamara/redaxo_yform/search?q=getCRSFKey&type=

PR mit Alias-Funktion gewünscht?

dergel added a commit that referenced this issue May 27, 2021
@dergel
Copy link
Member

dergel commented May 27, 2021

oh mann :) .. Sorry, aber nun passt es ..

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