Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Not Writing Out Remote IP in HTTP/3? #7939

Closed
FlyingWithJerome opened this issue Nov 1, 2021 · 0 comments
Closed

Not Writing Out Remote IP in HTTP/3? #7939

FlyingWithJerome opened this issue Nov 1, 2021 · 0 comments
Assignees

Comments

@FlyingWithJerome
Copy link

@FlyingWithJerome FlyingWithJerome commented Nov 1, 2021

I did this

curl --http3 -w 'Remote IP: %{remote_ip}, Remote Port: %{remote_port}\n' https://google.com --output /dev/null --silent
Remote IP: , Remote Port: 443

I would like to write out the remote IP address and port for this HTTP/3 transaction, but it seems that "%{remote_ip}" is not working?
I can see the IP address in the verbose mode, though, like * Trying 142.250.190.46:443... And without --http3 this write-out works perfect.

I expected the following

curl --http3 -w 'Remote IP: %{remote_ip}, Remote Port: %{remote_port}\n' https://google.com --output /dev/null --silent
Remote IP: 142.250.190.46, Remote Port: 443

curl/libcurl version

curl 7.80.0-DEV (x86_64-pc-linux-gnu) libcurl/7.80.0-DEV OpenSSL/3.0.0 zlib/1.2.11 zstd/1.4.4 ngtcp2/0.1.0-DEV nghttp3/0.1.0-DEV
Release-Date: [unreleased]
Protocols: dict file ftp ftps gopher gophers http https imap imaps mqtt pop3 pop3s rtsp smb smbs smtp smtps telnet tftp
Features: alt-svc AsynchDNS HSTS HTTP3 HTTPS-proxy IPv6 Largefile libz NTLM NTLM_WB SSL TLS-SRP UnixSockets zstd

operating system

Linux xxx 4.18.0-240.15.1.el8_3.x86_64 #1 SMP Wed Feb 3 03:12:15 EST 2021 x86_64 x86_64 x86_64 GNU/Linux
@bagder bagder self-assigned this Nov 2, 2021
bagder added a commit that referenced this issue Nov 2, 2021
So that CURLINFO_PRIMARY_IP etc work for HTTP/3 like for other HTTP
versions.

Reported-by: Jerome Mao
Fixes #7939
@bagder bagder closed this in f5ee9cf Nov 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

2 participants