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

Cobbler hides JSON validation errors. #4

Open
primarilysnark opened this issue Jan 16, 2017 · 0 comments
Open

Cobbler hides JSON validation errors. #4

primarilysnark opened this issue Jan 16, 2017 · 0 comments

Comments

@primarilysnark
Copy link

primarilysnark commented Jan 16, 2017

Cobbler hides JSON validation errors for resumés, instead assuming that the invalid file is a CSON-formatted resumé.

Example

Suppose the example resumé:

{
  "basics": 
}

The expected response would be:

Error: SyntaxError: Unexpected token } in JSON at position 15
    at JSON.parse (<anonymous>)
    at Promise.then.contents (/testdouble/cobbler/lib/cobbler.js:31:21)

Cobbler instead returns:

#/: Additional properties not allowed
    at Promise.then.then.then.catch.err (/testdouble/cobbler/lib/cobbler.js:47:17)
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

No branches or pull requests

1 participant