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.

Sign up for GitHub

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

Jump to bottom

feat(clustering): hybrid forward proxy https #9773

Merged
merged 7 commits into from
Nov 18, 2022
Merged

feat(clustering): hybrid forward proxy https #9773

merged 7 commits into from
Nov 18, 2022

Conversation

fffonion
Copy link
Contributor

Add support of talk to an HTTP tunnel in https

FTI-2996

@github-actions github-actions bot added core/cli core/clustering core/configuration core/templates chore Not part of the core functionality of kong, but still needed labels Nov 17, 2022
@fffonion fffonion force-pushed the feat/hybrid-forward-proxy-https branch from 5481a07 to 4671d43 Compare November 17, 2022 08:21
@pull-request-size pull-request-size bot added size/L and removed size/M labels Nov 17, 2022
@github-actions github-actions bot removed the core/cli label Nov 17, 2022
Base automatically changed from feat/hybrid-forward-proxy to master November 17, 2022 08:48
@fffonion fffonion force-pushed the feat/hybrid-forward-proxy-https branch from 4671d43 to ffd0bdf Compare November 17, 2022 09:47
@fffonion fffonion marked this pull request as ready for review November 17, 2022 09:47
@fffonion fffonion force-pushed the feat/hybrid-forward-proxy-https branch from ffd0bdf to 6bc3dc6 Compare November 17, 2022 09:50
dndx
dndx reviewed Nov 17, 2022
View reviewed changes
kong/resty/websocket/client.lua Outdated Show resolved Hide resolved
kong/clustering/utils.lua Show resolved Hide resolved
kong/resty/websocket/client.lua Show resolved Hide resolved
@github-actions github-actions bot removed the chore Not part of the core functionality of kong, but still needed label Nov 17, 2022
@guanlan guanlan added this to the 3.1 milestone Nov 17, 2022
@chronolaw chronolaw changed the title Feat/hybrid forward proxy https feat(clustering): hybrid forward proxy https Nov 18, 2022
dndx
dndx approved these changes Nov 18, 2022
View reviewed changes
Copy link
Member

@dndx dndx left a comment

Choose a reason for hiding this comment

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

This looks fine, can be merged after all comments are addressed.

@fffonion fffonion force-pushed the feat/hybrid-forward-proxy-https branch from a8f5c85 to 80986dc Compare November 18, 2022 06:09
@fffonion fffonion merged commit a275f39 into master Nov 18, 2022
@fffonion fffonion deleted the feat/hybrid-forward-proxy-https branch November 18, 2022 07:15
oowl pushed a commit that referenced this pull request Aug 15, 2024
@team-gateway-bot @chobits @chronolaw
style(dns): style and typo fixes for dns client (#9773)
0a4d8e0 
KAG-5001

two blank lines between blocks
optimize table.insert
family should compare to domain, not name.

Cherry-picked from #13389

---------

Co-authored-by: Xiaochen Wang <wangxiaochen0@gmail.com>
Co-authored-by: Chrono <chrono_cpp@me.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@chronolaw chronolaw chronolaw left review comments

@dndx dndx dndx approved these changes

Assignees
No one assigned
Labels
core/clustering core/configuration core/templates size/L
Projects
None yet
Milestone
3.1
Development

Successfully merging this pull request may close these issues.
4 participants
@fffonion @dndx @chronolaw @guanlan