Skip to content

Conversation

@bungle
Copy link
Member

@bungle bungle commented Oct 24, 2025

Summary

This is not needed anymore, thus removing.

It was needed in old OpenResty version, because there were limitations in OpenResty that were since fixed.

@bungle bungle requested a review from oowl October 24, 2025 12:55
@bungle bungle force-pushed the refactor/alpn branch 2 times, most recently from 5c53eb8 to f574151 Compare October 24, 2025 14:09
@bungle bungle force-pushed the refactor/alpn branch 2 times, most recently from 568d917 to 81c67fc Compare October 24, 2025 14:49
@bungle
Copy link
Member Author

bungle commented Oct 24, 2025

After this has been merged, we need to merge (and bump Kong module there):
Kong/kong#14782

After Kong PR is merged, we can revert the 81c67fc in this repo OR we can revert it before merging this and then merge here as red.

And then do the same here:
https://github.com/Kong/kong-ee/pull/14699

@bungle bungle requested a review from chronolaw October 24, 2025 18:14
Copy link
Contributor

@chronolaw chronolaw left a comment

Choose a reason for hiding this comment

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

Could we have an explanation for this PR? Why did we don't need this?

@bungle bungle merged commit 3f30591 into master Oct 25, 2025
8 checks passed
@bungle bungle deleted the refactor/alpn branch October 25, 2025 09:23
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.

4 participants