-
Notifications
You must be signed in to change notification settings - Fork 43
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: Invalid JSON (Unexpected " " at position 4 in state STOP) #103
Comments
Fails again, even with |
I don't think that's |
I double-checked in the I'll try downgrading for real and see if that gives us more stable results. Edit: We're back to the old issue: https://travis-ci.org/sumup/circuit-ui/jobs/559625735 |
Hmm, I checked the build you sent and its output is unexpected:
Running locally, I get something like:
|
We're still running into this with the latest version 2.3.0:
Any clues what's going on? |
we encountered this problem, turns out the CI environment we're running in (travis) was using npm 5.6.0 which didn't support |
We are also still seeing this issue with
Any help on hint would be very cool to have |
@quinnturner I see the same problem with
I tried looking into it a bit, but didn't get too far. The problem is, that
And I also found out, that |
@jzsplk Thanks for the report. Can you attach your dependencies or run |
This was the issue for us, using circleci, upgrading our build to use the circleci/node:12.16.1 docker image worked. |
Hi, I'm getting the same issue in my Dockerfile. It sometimes works and sometimes fails. Using version 12.16.1 On a side note... When I run
With the vast majority of them linked to "Validation Bypass". It seems to be a circular dependency. Validation Bypass is a dependency of jest. I'd like to note that this issue with low vulnerabilities with jest has been recurrent. |
I also see the same output as @kemicofa ( |
yarnpkg/yarn#7404 seems to be the core issue, with jestjs/jest#8682 possibly being the trigger The advisory that triggers this is https://www.npmjs.com/advisories/1490 |
I have the same problem I think
|
Switching back to my own yarn-audit-ci package to run yarn audit in CI. Does exactly what I need in CI: only blocks the CI on the configured severtity level of vulnerabilities found, in the most lightweight manner possible. You guys might wanna try it out. P.S. As to this issue, after doing some collaboration on the audit-ci project I belive moving towards parsing a heavy json output was a mistake, and even though we switched to JSONStream at some point it is still not good enough as the number of entries is high. The truth is we don't really care for a detailed output in CI, we can just need a fast go/no-go, and we can then do the investigation locally. |
This issue is certainly the biggest blocker to workflows for this project. Using @dubbha, I looked into your package and I found it's approach promising for basic use-cases. If a project only uses Yarn, and doesn't need to support I am open to ideas on how to make this issue less of a blocker. We could, for example, incorporate |
We were running into the same problem as described in the first post of this issue. After some debugging, I found out that the string in the buffer where the JSON decoding fails is The side effect of this change is that this process now has a significant memory footprint. |
I'm using the version 3.1.1. I've upgraded to |
@vctormb Try refreshing your |
We have the same issue with different versions of
Error message:
where .audit-ci.json:
@quinnturner Do you have any ideas on how to solve this problem? |
Hi @khaleksa, |
Hi @quinnturner, |
Hi @quinnturner, I'm glad that you are offering help on our project with the updated
Seems running |
Hey @crypto-matto, thanks for the report. What's the result when you use this command? yarn npm audit --recursive --json --all |
Hi @quinnturner, thanks for the suggestion. This is the result.
After I upgraded
Yet I'm facing a lot of other issues when dealing with migrating |
Due to: #96
(It's visible here: https://travis-ci.org/sumup/circuit-ui/jobs/559411807)
Related: #102
The text was updated successfully, but these errors were encountered: