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

Error in Example Demo #1049

Closed
dikaio opened this issue Jan 5, 2021 · 8 comments
Closed

Error in Example Demo #1049

dikaio opened this issue Jan 5, 2021 · 8 comments
Assignees
Labels
bug Something isn't working hacktoberfest-help-needed The maintainer needs help due to time constraint/missing knowledge priority Priority fix or enhancement

Comments

@dikaio
Copy link

dikaio commented Jan 5, 2021

The example demo currently produces an application error here while signed in. You need to be signed in to see the error.

@dikaio dikaio added the bug Something isn't working label Jan 5, 2021
@balazsorban44
Copy link
Member

balazsorban44 commented Jan 6, 2021

Could you please be a bit more verbose about what kind of error you are seeing, and how to reproduce it?

@balazsorban44 balazsorban44 added the incomplete Insufficient reproduction. Without more info, we won't take further actions/provide help. label Jan 6, 2021
@dikaio
Copy link
Author

dikaio commented Jan 7, 2021

Hi @balazsorban44. Sure.

Your example application https://github.com/nextauthjs/next-auth-example produces an application error on the protected-ssr page, you can see the error here.

To reproduce the error:

visit https://next-auth-example.now.sh/
sign in
visit https://next-auth-example.now.sh/protected-ssr

This will produce the error

502: BAD_GATEWAY
Code: NO_RESPONSE_FROM_FUNCTION
ID: sfo1::dpn2b-1609986547622-a9ca018084fa`

This error is not produced from cloning and using your repo, its the example application you guys have hosted.

@balazsorban44
Copy link
Member

Thanks! I am wondering if these are related:
#660, #1021

@balazsorban44 balazsorban44 added hacktoberfest-help-needed The maintainer needs help due to time constraint/missing knowledge and removed incomplete Insufficient reproduction. Without more info, we won't take further actions/provide help. labels Jan 7, 2021
@iaincollins iaincollins added the priority Priority fix or enhancement label Jan 7, 2021
@iaincollins
Copy link
Member

Thanks for raising this! I saw it earlier this week and came here to check to see if anyone had flagged it.

The demo actually uses an in memory database, so has no external dependencies (beyond OAuth provides).

The Protected SSR route and the JWT route are both not working, but were previously.

I think there may have been some sort of change on the Vercel platform that is triggering this issue.

@iaincollins iaincollins self-assigned this Jan 7, 2021
@iaincollins
Copy link
Member

I'm happy to pick this up and debug it - that's probably the best idea as I'm the only one who currently has access to the logs or to do deployments, as we don't have paid or open source team account yet.

I checked the logs and do have an error but it didn't seem helpful, so will likely create a custom page with some debugging.

It's seems likely the problem is with with decoding the JWT somehow, as both the JWT endpoint and the Protected endpoint use it. I have no idea why that would fail but it should be easy to debug and replicate by creating a simple API route that handles decoding the token and trapping any errors it throws.

@iaincollins
Copy link
Member

Okay nope, some incorrect breaking changes were made to the repo I was unaware of. I'm going to ask folks that not make changes the Example repo for now.

@iaincollins
Copy link
Member

Okies:

  • I've rolled out a fix for the JWT page and removed the SSR example entirely (it wasn't intended to be part of the live demo).
  • I've updated the example to Next.js 10.
  • I've closed the open PRs, and explained why in the case of the VERCEL_URL PR.

I think we should leave the exiting example repo alone for now.

Hopefully in the future we can have a wider range of examples (with appropriate tests and pre-live environments in place!)

@iaincollins
Copy link
Member

Closing as is resolved for now.

Thanks again for raising the issue!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working hacktoberfest-help-needed The maintainer needs help due to time constraint/missing knowledge priority Priority fix or enhancement
Projects
None yet
Development

No branches or pull requests

3 participants