Bump up version of http-proxy

movableink / doorman

HTTP Proxy + OAuth

MIT License

167 stars 52 forks source link

Bump up version of http-proxy #33

Closed AlexRRR closed 9 years ago

AlexRRR commented 9 years ago

I managed to track down issue #32, it was simply a matter of bumping up the version of http-proxy. I don't know the details of why but it seems it does not play well with util.inherits.

amokhtar commented 9 years ago

:+1:

inakianduaga commented 9 years ago

Thanks!

Until the PR is merged, in the meantime I'm manually updating all npm packages on my dockerized doorman image build before installing doorman packages

#Dockerfile
... 
RUN npm install -g npm-check-updates && \
         npm-check-updates -u && \ 
         npm install
...

pdehlke commented 9 years ago

Confirmed working fix, thanks! Looking forward to this getting merged.