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

feat: support ipv6 4-tuple (#724) #728

Merged
merged 1 commit into from
Feb 10, 2025
Merged

Conversation

chilli13
Copy link
Contributor

@chilli13 chilli13 commented Feb 8, 2025

@dosubot dosubot bot added the enhancement New feature or request label Feb 8, 2025
@chilli13 chilli13 closed this Feb 8, 2025
@chilli13 chilli13 reopened this Feb 8, 2025
@chilli13
Copy link
Contributor Author

chilli13 commented Feb 8, 2025

Test ok on x86 os :kernel 5.15.0(unbuntu 22.04), 4.19.90(ctyunos 2.0.1), 5.10.134(anolisos 8.8). Could u help
test on ubuntu 18.04 and arm os? thks @cfc4n


2025-02-08T11:09:17+08:00 ??? UUID:82542_82542_nginx_10_0_2001:db8::99:10_50874-2001:db8::99:11_5443, Name:HTTP2Request, Type:2, Length:316

Frame Type	=>	SETTINGS

Frame Type	=>	WINDOW_UPDATE

Frame Type	=>	HEADERS
header field ":method" = "GET"
header field ":path" = "/"
header field ":scheme" = "https"
header field ":authority" = "[2001:db8::99:11]:5443"
header field "user-agent" = "curl/7.71.1"
header field "accept" = "*/*"

Frame Type	=>	SETTINGS

2025/02/08 11:09:17 [http2 response] Dump HTTP2 Frame error: unexpected EOF
2025-02-08T11:09:17+08:00 ??? UUID:82542_82542_nginx_10_1_2001:db8::99:10_50874-2001:db8::99:11_5443, Name:HTTP2Response, Type:4, Length:1247

Frame Type	=>	SETTINGS

Frame Type	=>	WINDOW_UPDATE

Frame Type	=>	SETTINGS

Frame Type	=>	HEADERS
header field ":status" = "200"
header field "server" = "nginx"
header field "date" = "Sat, 08 Feb 2025 03:09:16 GMT"
header field "content-type" = "text/html"
header field "content-length" = "10492"
header field "last-modified" = "Mon, 06 Jan 2025 03:17:06 GMT"
header field "vary" = "Accept-Encoding"
header field "etag" = "\"677b4b32-28fc\""
header field "cache-control" = "private, no-store, no-cache, must-revalidate, proxy-revalidate"
header field "x-frame-options" = "SAMEORIGIN"
header field "x-content-type-options" = "nosniff"
header field "x-xss-protection" = "1; mode=block"
header field "strict-transport-security" = "max-age=31536000; includeSubdomains;"
header field "x-permitted-cross-domain-policies" = "none"
header field "referrer-policy" = "strict-origin-when-cross-origin"
header field "content-security-policy" = "script-src 'self' 'unsafe-inline' 'unsafe-eval' data: blob:;img-src  'self' 'unsafe-inline' data: blob:; style-src 'self' 'unsafe-inline'; worker-src 'self' 'unsafe-inline' * blob:; font-src 'self' 'unsafe-inline' data: blob:;"
header field "x-downlaod-options" = "noopen"
header field "accept-ranges" = "bytes"

2025-02-08T11:09:25+08:00 ??? UUID:82542_82542_nginx_10_0_127.0.0.1_52904-127.0.0.1_5443, Name:HTTPRequest, Type:1, Length:574
POST /posidon/v1/api/monitor/captureAndSend HTTP/1.1
Host: 127.0.0.1:5443
Accept-Encoding: gzip,deflate
Connection: Keep-Alive
Content-Length: 294
Content-Type: application/json; charset=UTF-8
User-Agent: Apache-HttpClient/4.5.14 (Java/1.8.0_301)


2025-02-08T02:27:46Z ??? UUID:432952_432952_curl_5_1_2001:db8::200_35740-2001:db8::200_443, Name:HTTPRequest, Type:1, Length:84
GET /1.txt HTTP/1.1
Host: [2001:db8::200]
Accept: */*
User-Agent: curl/7.81.0


2025-02-08T02:27:46Z ??? UUID:949_949_nginx_5_0_2001:db8::200_35740-2001:db8::200_443, Name:HTTPRequest, Type:1, Length:84
GET /1.txt HTTP/1.1
Host: [2001:db8::200]
Accept: */*
User-Agent: curl/7.81.0

2025-02-08T02:27:46Z ??? UUID:949_949_nginx_5_1_192.168.10.122_44688-192.168.10.122_443, Name:HTTPResponse, Type:3, Length:247
HTTP/1.1 200 OK
Content-Length: 3
Accept-Ranges: bytes
Connection: keep-alive
Content-Type: text/plain
Date: Sat, 08 Feb 2025 02:27:45 GMT
Etag: "67626c3e-3"
Last-Modified: Wed, 18 Dec 2024 06:31:26 GMT
Server: nginx/1.18.0 (Ubuntu)

123
2025-02-08T02:27:46Z ??? UUID:432951_432951_curl_5_1_192.168.10.122_44688-192.168.10.122_443, Name:HTTPRequest, Type:1, Length:83
GET /1.txt HTTP/1.1
Host: 192.168.10.122
Accept: */*
User-Agent: curl/7.81.0

@cfc4n cfc4n requested a review from Asphaltt February 8, 2025 10:48
Copy link
Member

@Asphaltt Asphaltt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

Leave two nit comments below.

user/event/event_openssl.go Outdated Show resolved Hide resolved
user/event/event_openssl.go Outdated Show resolved Hide resolved
@chilli13 chilli13 force-pushed the feat/ipv6-tuple branch 2 times, most recently from aeccdd4 to 34af534 Compare February 10, 2025 00:01
Copy link
Member

@Asphaltt Asphaltt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

Copy link
Member

@cfc4n cfc4n left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, Thanks.

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Feb 10, 2025
@cfc4n cfc4n merged commit dcfc3cf into gojue:master Feb 10, 2025
6 checks passed
@chilli13 chilli13 deleted the feat/ipv6-tuple branch February 11, 2025 00:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request lgtm This PR has been approved by a maintainer
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants