-
Notifications
You must be signed in to change notification settings - Fork 417
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
Google and Firebase - Unexpected end of JSON input #309
Comments
Hi @gastonmorixe, thanks for reporting. According to the stacktrace, the plugin crashed, because it got an invalid output from the Can you try to execute it manually and post the output, so that we can check what exactly is going wrong there? |
@HyperBrain also
|
@HyperBrain sorry to add more comments, it seems that with
|
Thanks a lot. It seems that exactly the same happens during execution of the plugin (there might be a bug in the plugin that prevents noticing the npm error and so the parser fails in the end). I will try if I get some time tomorrow, so that I can prepare exactly the same package.json, so that I can try to find out why npm fails here. (currently the plugin is based entirely on npm internally). I'll let you know if I found out anything. |
I moved to |
Hey, I believe I might be having the same issue. Works fine running locally with $sls offline start but once I try to deploy it I get that same error. Any ideas on how to tackle this issue?
Offline version
package.json
|
Looks like this is caused by using yarn. |
Just got caught by this while using |
@shwanton The section has been added just a few days ago ;-) |
I"m having this problem without using yarn, with identical results to @alopes |
@samwiseduzer Can you check your npm version and try to delete the npm cache (see |
Deleting and reinstalling my packages seemed to take care of this |
`nodegit` caused all sorts of issues, and it takes too long to compile whether locally or on CI. Even worse, due to [a bug in `serverless-webpack`](serverless-heaven/serverless-webpack#309) we have to compile it twice, which takes over 15 minutes on Travis. This PR drops `nodegit` and replaces it with a regular `git` binary [compiled specifically for AWS Lambda](https://github.com/pimterry/lambda-git). This may or may not work!
`nodegit` caused all sorts of issues, and it takes too long to compile whether locally or on CI. Even worse, due to [a bug in `serverless-webpack`](serverless-heaven/serverless-webpack#309) we have to compile it twice, which takes over 15 minutes on Travis. This PR drops `nodegit` and replaces it with a regular `git` binary [compiled specifically for AWS Lambda](https://github.com/pimterry/lambda-git). This may or may not work!
`nodegit` caused all sorts of issues, and it takes too long to compile whether locally or on CI. Even worse, due to [a bug in `serverless-webpack`](serverless-heaven/serverless-webpack#309) we have to compile it twice, which takes over 15 minutes on Travis. This PR drops `nodegit` and replaces it with a regular `git` binary [compiled specifically for AWS Lambda](https://github.com/pimterry/lambda-git). This may or may not work!
I ran into this problem again because of unmet peer dependencies (which I discovered after running "npm ls -prod -json -depth=1"). Fixing the peer dependency took care of the issue for me. |
@alopes Yarn support is available in |
Update: Yarn support is just released with 5.1.0 💥 |
@gastonmorixe Is there any update for this issue on your side. In case you use yarn, you could try plugin version 5.1.0 and configure it to use yarn as packager (if you don't use it, you can try that too ;-) ). |
Sorry guys, not using it anymore. Just using firebase directly.
On Wednesday, March 14, 2018, Frank Schmid ***@***.***> wrote:
@gastonmorixe Is there any update for this issue on your side. In case
you use yarn, you could try plugin version 5.1.0 (if you don't use it, you
can try that too ;-) ).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.<
…--
Sent from Gmail Mobile
|
Ok. I'm closing the issue now. Feel free to continue here if needed 😄 |
@gastonmorixe @alopes I finally found the root cause for the original The fix will be released soon with version 5.1.2. If you have some time, you can check out if the current master branch works for you. |
This is a (Bug Report / Feature Proposal)
Bug
Description
For bug reports:
Deployed correctly
Additional Data
package.json
The text was updated successfully, but these errors were encountered: