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

fix: support ip:port in host #139

Merged
merged 10 commits into from
Jan 12, 2024
Merged

Conversation

theweakgod
Copy link
Contributor

@theweakgod theweakgod commented Nov 28, 2023

support ip:port in host
#138
#10531

@CLAassistant
Copy link

CLAassistant commented Nov 28, 2023

CLA assistant check
All committers have signed the CLA.

lib/resty/radixtree.lua Show resolved Hide resolved
lib/resty/radixtree.lua Show resolved Hide resolved
@monkeyDluffy6017
Copy link

I will check this later, thanks for your contribution!

@monkeyDluffy6017
Copy link

@theweakgod are you stilling working on the pr?

@theweakgod
Copy link
Contributor Author

@theweakgod are you stilling working on the pr?

Sure

@monkeyDluffy6017
Copy link

@theweakgod please resolve the review comments

@theweakgod
Copy link
Contributor Author

theweakgod commented Jan 5, 2024

@theweakgod please resolve the review comments

@monkeyDluffy6017 What should I do, I have replied

@theweakgod
Copy link
Contributor Author

@theweakgod please resolve the review comments

Ok, I got it

@monkeyDluffy6017
Copy link

@theweakgod I didn't see your reply
image

@theweakgod
Copy link
Contributor Author

@theweakgod I didn't see your reply image

image But I really replied.

@monkeyDluffy6017
Copy link

The comment is still pending, please check the button at the upright corner
image

@theweakgod
Copy link
Contributor Author

The comment is still pending, please check the button at the upright corner image

Okay, thank you, this is my first time using this.

@monkeyDluffy6017 monkeyDluffy6017 merged commit ee3e37b into api7:master Jan 12, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants