camicroscope / Caracal

Conslidated Backend, Auth, and Security Services for caMicroscope
GNU General Public License v3.0
15 stars 94 forks source link

circular dependency warnings #53

Open birm opened 4 years ago

birm commented 4 years ago
ca-back     | (node:28) Warning: Accessing non-existent property 'count' of module exports inside circular dependency
ca-back     | (Use `node --trace-warnings ...` to show where the warning was created)
ca-back     | (node:28) Warning: Accessing non-existent property 'findOne' of module exports inside circular dependency
ca-back     | (node:28) Warning: Accessing non-existent property 'remove' of module exports inside circular dependency
ca-back     | (node:28) Warning: Accessing non-existent property 'updateOne' of module exports inside circular dependency

@ develop

akhil-rana commented 4 years ago

This seems like a problem with mongo db as mentioned here too. Since we've updated node to 14.0 it seems to trigger this now. Updating mongodb might be a solution.

siddhanth339 commented 3 years ago

Running npm update solved the issue. So, I think adding npm update after this line in Caracal/Dockerfile will solve the problem. Should I make a pull request with the above change?

birm commented 3 years ago

I'm not a big fan of having the dockerfile update, it's good to have predictable dependencies a la package lock. I've updated the dependencies though - https://github.com/camicroscope/Caracal/commit/faa16df98a86ff29aeff4f3044689457a6915f77