Closed Shinyzenith closed 2 years ago
I went with logrus as the logger, please do let me know if you guys would like some other package to handle logging.
I believe the daemons use zap, might be best to use the same logger between go services?
Alright, I'll use zap.
Currently the database connection is stored as a singleton global in memory. Should we do the same for collections too? currently we only have two db.Colection calls but I can totally see it scaling up in the future.
The above ^ also goes for the connection context...should we use singleton pattern for it too?
The above ^ also goes for the connection context...should we use singleton pattern for it too?
Some interaction from anyone at team would help me to get along with the patch, making all the decisions alone might lead to unwanted design patterns which will lead to unnecessary refactors later.
@angelofallars feel free to join in since you're interested in working with me on the patch.
Yeah, I'll join in too!
Closing due to lack of interaction.
You should consider forking the project. Sorry that you didn't have more feedback from the devs on this.
TODO:
api.go
aka the main file should only have the routing and call the handler functions from the logic package.Signed-off-by: Shinyzenith aakashsensharma@gmail.com