Skip to content

Commit 06f13e9

Browse files
committed
fix(module): use useragent_ip instead of conn client_ip, should have more correct information
1 parent c2f2990 commit 06f13e9

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

src/redirectionio_protocol.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -84,7 +84,7 @@ apr_status_t redirectionio_protocol_match(redirectionio_connection *conn, redire
8484
return APR_EGENERAL;
8585
}
8686

87-
redirectionio_request_set_remote_addr(ctx->request, r->connection->client_ip, config->trusted_proxies);
87+
redirectionio_request_set_remote_addr(ctx->request, r->useragent_ip, config->trusted_proxies);
8888

8989
// Serialize request
9090
request_serialized = redirectionio_request_json_serialize(ctx->request);
@@ -166,7 +166,7 @@ apr_status_t redirectionio_protocol_log(redirectionio_connection *conn, redirect
166166
response = response->next;
167167
}
168168

169-
log = redirectionio_api_create_log_in_json(ctx->request, response->status, ctx->response_headers, ctx->action, PROXY_VERSION_STR(PROXY_VERSION), response->request_time / 1000, ctx->action_match_time, ctx->proxy_response_time, r->connection->client_ip);
169+
log = redirectionio_api_create_log_in_json(ctx->request, response->status, ctx->response_headers, ctx->action, PROXY_VERSION_STR(PROXY_VERSION), response->request_time / 1000, ctx->action_match_time, ctx->proxy_response_time, r->useragent_ip);
170170

171171
if (log == NULL) {
172172
return APR_EGENERAL;

0 commit comments

Comments
 (0)