Skip to content

fix(sessions): return non-nil Session when store.New encounters error - #300

Open
AdamMagued wants to merge 1 commit into
gorilla:mainfrom
AdamMagued:fix-issue-288
Open

AdamMagued wants to merge 1 commit into
gorilla:mainfrom
AdamMagued:fix-issue-288

Conversation

@AdamMagued

Copy link
Copy Markdown

Fixes #288

Summary

  • Return a valid Session object when store.New encounters an error, avoiding nil pointer panics in callers.
  • Add unit tests covering error scenarios from custom and cookie stores.

Verified via container sandbox test suite.

Signed-off-by: AdamMagued <adamismailmageud@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Panic if store.New returns an error

1 participant