-
Notifications
You must be signed in to change notification settings - Fork 123
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 Message from repository.getComponentInfo logged with [Object object] #789
Comments
Thanks for the issue @bassettsj - I am investigating. |
@matteofigus I added a failing unit test showing the error. I will open the PR, sorry if this duplicating work. |
…bject] - add new test case reproducing the error
No problem at all @bassettsj I was looking at the code but didn't write any code yet. If you would want to attempt a PR for the tests and a fix, that would super fantastic, want to give it a go? |
…bject] - add new test case reproducing the error
Add test for issue #789 showing error message [Object object]
Hi I've been trying to use the starter kit to create my first registry but I'm only seeing the JSON not valid error above - could you point me in any direction of what's going wrong? See example oc-client error which occurs on standard example of starter kit deployment https://starter-kit-itrqyqhfdt.now.sh/oc-client/0.42.16/~info |
fixed thanks! |
Who is the bug affecting?
Registry maintainers
What is affected by this bug?
The error message is not very helpful.
When does this occur?
repository.geComponentInfo
's callback is called with a non error object from a storage error it does not give a helpful message"component not available: [object Object]"
Where on the platform does it happen?
api
How do we replicate the issue?
https://demo-oc-registry.herokuapp.com/tommys-component/1.0.1/~preview
Expected behavior (i.e. solution)
Expected to give the actual error message which is:
{ code: 'file_not_valid', msg: 'File "components/tommys-component/1.0.0/package.json" not valid' }
What version of OC, Node.js and OS are you using?
Other Comments
The text was updated successfully, but these errors were encountered: