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

add v6 support for AddMissingPort function #203

Merged
merged 1 commit into from
Aug 19, 2022

Conversation

welkeyever
Copy link
Member

What type of PR is this?

fix

What this PR does / why we need it (English/Chinese):

EN: add support for bare v6 address in AddMissingPort function
ZH:AddMissingPort函数增加对裸v6地址的处理

Change-Id: Ib219227e0813733bd7f1fbe950dedd3cf21b042c
@codecov
Copy link

codecov bot commented Aug 18, 2022

Codecov Report

Merging #203 (94bebf8) into develop (8697481) will decrease coverage by 0.04%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop     #203      +/-   ##
===========================================
- Coverage    59.03%   58.98%   -0.05%     
===========================================
  Files           79       79              
  Lines         7997     7998       +1     
===========================================
- Hits          4721     4718       -3     
- Misses        2928     2932       +4     
  Partials       348      348              
Impacted Files Coverage Δ
pkg/common/utils/path.go 82.60% <100.00%> (+0.51%) ⬆️
pkg/common/timer/timer.go 61.90% <0.00%> (-19.05%) ⬇️
pkg/network/standard/buffer.go 81.08% <0.00%> (-0.50%) ⬇️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@welkeyever welkeyever merged commit 9f2cacd into cloudwego:develop Aug 19, 2022
@welkeyever welkeyever deleted the fix/add_missing_port_in_v6 branch August 19, 2022 02:41
FGYFFFF pushed a commit to FGYFFFF/hertz that referenced this pull request Aug 24, 2022
fix: add v6 support for AddMissingPort function
FGYFFFF pushed a commit to FGYFFFF/hertz that referenced this pull request Aug 24, 2022
fix: add v6 support for AddMissingPort function
FGYFFFF pushed a commit to FGYFFFF/hertz that referenced this pull request Aug 24, 2022
fix: add v6 support for AddMissingPort function
FGYFFFF pushed a commit that referenced this pull request Aug 25, 2022
fix: add v6 support for AddMissingPort function
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants