fastify-oauth2 icon indicating copy to clipboard operation
fastify-oauth2 copied to clipboard
verified publisher icon fastify

Misleading documentation for startRedirectPath

Open johnkmzhou opened this issue 1 year ago • 1 comments

Prerequisites

  • [X] I have written a descriptive issue title
  • [X] I have searched existing issues to ensure the issue has not already been raised

Issue

In the README, under Usage, running the code will result in an error that states Method 'GET' already declared for route '/login/facebook'. The documentation should mention that the /login/facebook route only needs to be created if startRedirectPath has not been provided.

johnkmzhou avatar Jun 17 '24 21:06 johnkmzhou

Thanks for reporting! Would you like to send a Pull Request to address this issue? Remember to add unit tests.

mcollina avatar Jun 18 '24 09:06 mcollina