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

Forms using Blade and the @sessionToken directive (still) don't work properly #248

Open
SebKay opened this issue Dec 13, 2023 · 2 comments
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed

Comments

@SebKay
Copy link

SebKay commented Dec 13, 2023

I recently opened an issue about this, #239. After the v19.2.0 update this appeared to be fixed, however it isn't completely solved.

The problem is now intermittent. It seems to be fixed now when using a redirect helper such as back(), however not when using the redirect()->tokenRedirect() helper.

I've spent some time trying to figure out why, but I couldn't for the life of me figure out why it's happening.

@SebKay SebKay added bug Something isn't working unconfirmed Bug has not been reproduced yet labels Dec 13, 2023
@Kyon147
Copy link
Owner

Kyon147 commented Dec 13, 2023

Thanks for submitting and for the initial investigation - I'll take a look over Christmas when I have some time and see if I can find the issue.

@Kyon147
Copy link
Owner

Kyon147 commented May 1, 2024

Just an update here, I think the issue is host being missed off from the helper. I dont have time to fix at the moment but a short term fix, it to pass the host with the form submit as an input.

<input hidden name="host" value="{{\Illuminate\Support\Facades\Request::get('host')}}">

Ofcourse, like always you must pass host across all pages anyway. So make sure this also is happening.

@Kyon147 Kyon147 added enhancement New feature or request help wanted Extra attention is needed good first issue Good for newcomers and removed bug Something isn't working unconfirmed Bug has not been reproduced yet labels Sep 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants