Enhance the request object with the client_ip of the incoming connection

[aclodhopper]

The current code issues a debug message containing the "client_ip" of the incoming request already.
It would be useful to include the client ip of the request into the standard request object passed to the handlers - this way, handler code could easily determine and use the IP of the request client.

[DavesCodeMusings]

This should be fairly easy to accomodate.

In the function async def on_connect(self, reader, writer):, this...

client_ip = writer.get_extra_info('peername')[0]
if self.debug:
    print(f'Connection from client: {client_ip}')

try:
    req_buffer = await reader.read(self.req_buffer_size)
    req = await Thimble.parse_http_request(req_buffer)
    if self.debug:
        print(f'Request: {req}')
except Exception as ex:
    await self.send_error(400, writer)
    print(f'Unable to parse request: {ex}')
else:
    route_value = self.resolve_route(req['method'] + req['path'])

Would become this...

remote_addr = writer.get_extra_info('peername')[0]
if self.debug:
    print(f'Connection from client: {remote_addr}')

try:
    req_buffer = await reader.read(self.req_buffer_size)
    req = await Thimble.parse_http_request(req_buffer)
except Exception as ex:
    await self.send_error(400, writer)
    print(f'Unable to parse request: {ex}')
else:
    req['remote_addr'] = remote_addr
    if self.debug:
        print(f'Request: {req}')
    route_value = self.resolve_route(req['method'] + req['path'])

Changing client_ip to remote_addr to better match up with what Flask, NodeJS, PHP, etc. call it.

[DavesCodeMusings]

Added remote_addr to req to make the client's IP address available for use on the microcontroller side. If this is a feature you are using, please report back here with your success / failure.