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

Add option to skip cookie verification in middleware #312

Closed
wants to merge 1 commit into from

Conversation

4cecoder
Copy link

@4cecoder 4cecoder commented Aug 8, 2024

Related to #129

Add an option to skip cookie verification in the middleware.

  • Add a new Options struct to hold middleware options and a WithSkipCookieVerification function to set the SkipCookieVerification option in clerk/middleware_v2.go.
  • Modify the WithSessionV2 function in clerk/middleware_v2.go to check for the WithSkipCookieVerification option and skip cookie verification if set.
  • Add tests in clerk/middleware_v2_test.go to verify the functionality of the WithSkipCookieVerification option, covering both scenarios: with and without the option.
  • Update the example in examples/middleware/main.go to demonstrate the usage of the WithSkipCookieVerification option.

For more details, open the Copilot Workspace session.

Related to #129

Add an option to skip cookie verification in the middleware.

* Add a new `Options` struct to hold middleware options and a `WithSkipCookieVerification` function to set the `SkipCookieVerification` option in `clerk/middleware_v2.go`.
* Modify the `WithSessionV2` function in `clerk/middleware_v2.go` to check for the `WithSkipCookieVerification` option and skip cookie verification if set.
* Add tests in `clerk/middleware_v2_test.go` to verify the functionality of the `WithSkipCookieVerification` option, covering both scenarios: with and without the option.
* Update the example in `examples/middleware/main.go` to demonstrate the usage of the `WithSkipCookieVerification` option.

---

For more details, open the [Copilot Workspace session](https://copilot-workspace.githubnext.com/clerk/clerk-sdk-go/issues/129?shareId=XXXX-XXXX-XXXX-XXXX).
@4cecoder 4cecoder requested a review from a team as a code owner August 8, 2024 02:59
@4cecoder 4cecoder closed this by deleting the head repository Dec 6, 2024
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.

1 participant