wankdanker / node-odbc

ODBC bindings for node
MIT License
173 stars 79 forks source link

express-http-context does not work well with odbc #37

Open ranjithchev opened 6 years ago

ranjithchev commented 6 years ago

Hi

The express-http-context seems to be working well with most of the npm packages we are using but odbc. When I checked with them it looks like some changes might be required in the odbc package to support this.

Please take a look at the below link where I have mentioned that issue and the response I have from the owner of express-http-context npm module.

https://github.com/skonves/express-http-context/issues/18

wankdanker commented 6 years ago

Hi @ranjithchev, Thanks for reporting this. I've only seen express-http-context in passing but it did pique my interest. After looking a little bit into this issue, I've discovered that I don't know enough about cls, cls-hooked, or async_hooks to be able to take a stab at this quickly.

I tried to follow some of the other issues, but didn't really find a good example that would help me to adapt odbc for this. Hopefully someone else can help out.