Conclusion is to not use thread-local storage. Instead:
[ ] create a requestContext object that gets initialized in main-app. This will have getRequestId() and getUserId() and getAppSource(), as well as references to request and response
[ ] pass this object around everywhere
[ ] db.js should not expose an alexaDb or googleDb, but rather just accept the requestContext and choose one accordingly. That way, it doesn't need to get passed around as well
[x] logger.js should also just accept a requestContext, event and data
bigpopakap
commented
7 years ago
Conclusion is to not use thread-local storage. Instead:
requestContextobject that gets initialized inmain-app. This will havegetRequestId()andgetUserId()andgetAppSource(), as well as references torequestandresponsedb.jsshould not expose analexaDborgoogleDb, but rather just accept therequestContextand choose one accordingly. That way, it doesn't need to get passed around as welllogger.jsshould also just accept arequestContext,eventanddata