Douglas Wilson

Just to answer the latter part, you can use any mechanism you need to get the proxy IP, you are not limited to x-forwarded-for only. Just define your own custom...

Yes, I fixed that a long time ago, also in this repo too, which is probably why you cannot reproduce it while running tests locally from clone?

Oh, interesting. I guess that is true, if citgm uses published vereion it wouldn't be there. I was confused bc I saw the log saying downloading from github.com. I guess...

Oh, ok, good thing you mentioned! I wqs gonna try and squeeze it in today so we could get the citgm checked off, but I can hold off for us...

This was just to make a patch for 4 for citgm to use. Happy to do whatever. I have to leave in about 45 mins, and was on to do...

I'm going to close this due to both no response and it seems the change is copying the existing code into itself.

Typically the merge will also update the needed deps that also have differring majors between 4 and 5 so the fixes and changes listed as done in 4 will actually...

We can discuss once I get back next week :) I pretty much have this particular merge done too as I wqs working on it a couple weeks ago.

What is the npmCommand warnings referring to in that report output?

LGTM. As soon as the patch gets released within the next couple days I'll make the minor branch and land this there per usual.