-
-
Notifications
You must be signed in to change notification settings - Fork 115
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
Include "host" in @sessionToken output #194
Include "host" in @sessionToken output #194
Conversation
@michaellehmkuhl could you just take a look at this for PHP 8.0 failing? |
@Kyon147 it looks like it's the phpstan static code analysis step that's failing. That check seems to be disabled in the other package tests. It's only failing in this case because phpstan isn't able to resolve the |
@Kyon147 it crossed my mind that we could also probably get over the phpstan hurdle by making the class path explicit with a Unfortunately, it doesn't look like that approach made phpstan happy, either. |
@Kyon147 OK. Switched to the |
@michaellehmkuhl thanks for taking the time to look at this and get the CI/CD happy. I've merged this in now, so will look at pushing out a patch release this week as it should be helpful to people using Blade. |
Since the
URL::tokenRoute()
helper now includes thehost
parameter, it seems useful to include the same in the@sessionToken
blade directive output for forms.