express-ipfilter icon indicating copy to clipboard operation
express-ipfilter copied to clipboard

Move default getClientIp to use proxy-addr, just like express does.

Open pdefreitas opened this issue 6 years ago • 9 comments

Includes #61 which bumps lodash version to fix security issues. Typos fixed in the readme. Removes allowedHeaders in favour of proxy-addr (to streamline with Express behind proxies). For more info and discussion check issue #62

pdefreitas avatar Aug 29 '18 10:08 pdefreitas

@pdefreitas perhaps it would be good to republish as express-ipfilter2 ? since this is unmaintained?

jetersen avatar Sep 15 '18 08:09 jetersen

@baminteractive @coopman @nsbingham do you intend to maintain the package or could someone take ownership or at least be maintainer?

jetersen avatar Oct 22 '18 17:10 jetersen

@casz I can republish this project as express-ipfilter2 tomorow. I don't think the current owner of the repo is active.

pdefreitas avatar Oct 31 '18 09:10 pdefreitas

@pdefreitas could you try ask NPM transferring ownership first 😅

jetersen avatar Oct 31 '18 09:10 jetersen

https://www.npmjs.com/policies/disputes

jetersen avatar Oct 31 '18 09:10 jetersen

@casz thanks for sharing, I have contacted NPM and the NPM project owner.

pdefreitas avatar Oct 31 '18 10:10 pdefreitas

I followed up with @pdefreitas but just so every one knows, I left the company a while back so I haven't been able to administer this at all. I reached out to see if I can get the ownership transferred and will let you know what happens

ryanbillingsley avatar Oct 31 '18 14:10 ryanbillingsley

thanks for the update @ryanbillingsley 🙌

jetersen avatar Oct 31 '18 14:10 jetersen

@pdefreitas @casz as you may have noticed, the repo was transferred back to me. I will look at this PR as soon as I can this evening.

I mentioned it on #61, but I do want to get that version in before pushing a new major version out so I will try and get that taken care of if it isn't already by the time I can get to it.

ryanbillingsley avatar Oct 31 '18 19:10 ryanbillingsley