weizhouapache commented on PR #9534: URL: https://github.com/apache/cloudstack/pull/9534#issuecomment-2342078465
> > `client_address` is always `NULL` in my testing. > > I tested the PR again, following the steps provided on the [description](https://github.com/apache/cloudstack/pull/9534#issue-2468189305), and the IP addresses were stored correctly. > > It's important to remember that if there were any running CPVMs prior to the environment update, then they would need to be recreated in order to test if the `client_address` column is being populated correctly. > > Furthermore, it's interesting taking a look at the CPVM logs. The following logs were generated during my tests, which indicates that the client IP `192.168.122.1` was captured correctly. > > ```bash > 2024-09-10T17:19:38,330 INFO [cloud.consoleproxy.ConsoleProxyNoVNCHandler] (qtp2090537387-40:[]) Verifying session > source IP 192.168.122.1 from WebSocket connection request. > > 2024-09-10T17:19:38,331 DEBUG [cloud.consoleproxy.ConsoleProxyNoVNCHandler] (qtp2090537387-40:[]) Session source IP > 192.168.122.1 has been verified successfully. > ``` > > @weizhouapache, if the problem continues happening, could you please describe your tests steps and share the CPVM logs? Thanks @bernardodemarco It works after patching the cpvm -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
