Preserve client address down to Request.t

[vphantom]

It was as difficult as predicted, and I'm pretty sure you won't like it as-is, but this does preserve the client address, when available, down to Request.t.

[c-cube]

Ok so, why is the address optional in Request.t ? :)

[vphantom]

Ok so, why is the address optional in Request.t ? :)

Because Internal_ does not use an address.

[c-cube]

But internal is just for testing purpose, it could take a dummy address (e.g. loopback)

[vphantom]

I haven't found anyone actually using it so I didn't dare touch it. Is it related to that (*$R comment? Is that comment actually test code?

If so, then do I hard-code localhost in that comment or in Internal_ itself? Author's choice. 😉

[vphantom]

Option gone, localhost hard-coded in the test case. How's it look now?

[c-cube]

merged manually.