Skip to content

Session should generate on login#2

Merged
stevebauman merged 1 commit intoDirectoryTree:masterfrom
nick-potts:master
Apr 5, 2024
Merged

Session should generate on login#2
stevebauman merged 1 commit intoDirectoryTree:masterfrom
nick-potts:master

Conversation

@nick-potts
Copy link
Copy Markdown
Contributor

No description provided.

@nick-potts
Copy link
Copy Markdown
Contributor Author

fixes #1

@stevebauman
Copy link
Copy Markdown
Member

stevebauman commented Apr 4, 2024

@stevebauman
Copy link
Copy Markdown
Member

Confirmed that this isn't needed as Laravel already handles this under the hood.

@nick-potts
Copy link
Copy Markdown
Contributor Author

Laravel does the session automatically, but not the CSRF token - which is covered by this.

Not a huge issue, but laravel does recommend it in the docs:
https://laravel.com/docs/11.x/authentication#authenticating-users

@stevebauman stevebauman reopened this Apr 5, 2024
@stevebauman stevebauman merged commit d8ce2de into DirectoryTree:master Apr 5, 2024
@stevebauman
Copy link
Copy Markdown
Member

Thanks for the link @nick-potts! News to me, had no idea, appreciate the PR 🙏

stevebauman added a commit that referenced this pull request Jun 4, 2024
Bug 10 - Argument #2 ($user) must be of type Laravel\Socialite\Two\User
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

Successfully merging this pull request may close these issues.

2 participants