amida-tech / api-boilerplate

Node ES6 Express REST API boilerplate with Postgres Sequelize and Gulp
Apache License 2.0
153 stars 57 forks source link

An error occured [Circular] #2

Closed muddaser closed 6 years ago

muddaser commented 7 years ago

I ran the project as per instructions, the server starts on port 4000 and gives this error:

An error occured [Circular]

Here's the log if I try to access a route like /health-check etc.

"message":"Not Found","stack":"Error: API not found\n at APIError.ExtendableError (/home/balderdash/Projects/Blockgen New/central-backend-test/dist/server/helpers/APIError.js:35:15)\n at new APIError (/home/balderdash/Projects/Blockgen New/central-backend-test/dist/server/helpers/APIError.js:63:105)\n at /home/balderdash/Projects/Blockgen New/central-backend-test/dist/config/express.js:121:15\n at Layer.handle [as handle_request] (/home/balderdash/Projects/Blockgen New/central-backend-test/node_modules/express/lib/router/layer.js:95:5)\n at trim_prefix (/home/balderdash/Projects/Blockgen New/central-backend-test/node_modules/express/lib/router/index.js:312:13)\n at /home/balderdash/Projects/Blockgen New/central-backend-test/node_modules/express/lib/router/index.js:280:7\n at Function.process_params (/home/balderdash/Projects/Blockgen New/central-backend-test/node_modules/express/lib/router/index.js:330:12)\n at next (/home/balderdash/Projects/Blockgen New/central-backend-test/node_modules/express/lib/router/index.js:271:10)\n at Layer.handle [as handle_request] (/home/balderdash/Projects/Blockgen New/central-backend-test/node_modules/express/lib/router/layer.js:91:12)\n at trim_prefix (/home/balderdash/Projects/Blockgen New/central-backend-test/node_modules/express/lib/router/index.js:312:13)\n at /home/balderdash/Projects/Blockgen New/central-backend-test/node_modules/express/lib/router/index.js:280:7\n at Function.process_params (/home/balderdash/Projects/Blockgen New/central-backend-test/node_modules/express/lib/router/index.js:330:12)\n at next (/home/balderdash/Projects/Blockgen New/central-backend-test/node_modules/express/lib/router/index.js:271:10)\n at /home/balderdash/Projects/Blockgen New/central-backend-test/node_modules/express-winston/index.js:306:9\n at Layer.handle [as handle_request] (/home/balderdash/Projects/Blockgen New/central-backend-test/node_modules/express/lib/router/layer.js:95:5)\n at trim_prefix (/home/balderdash/Projects/Blockgen New/central-backend-test/node_modules/express/lib/router/index.js:312:13)"}

muddaser commented 7 years ago

I tried running the tests with yarn test. Got 15 passing.

Akronae commented 6 years ago

Same here

unprofessional commented 6 years ago

You may have neglected to use /api/... in the path. If I use /api/health-check I get back a 200 OK response.

Tenemo commented 6 years ago

I'm having a similar issue, I get the [Circular] error during sequelize.sync, routes work okay though.

unprofessional commented 6 years ago

We've pushed a host of updates which should fix the [Circular] problem (there should have been a .catch() statement for an unresolved promise when initializing sequelize). Likewise, we updated some outdated packages due to security vulnerabilities and have added API versioning in the path (configured in your .env -- see the .env.example.