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(router): http headers sensitive for expressions and insensitive for traditional_compatible #11905

Merged
merged 25 commits into from
Nov 8, 2023

Conversation

chronolaw
Copy link
Contributor

@chronolaw chronolaw commented Nov 2, 2023

Summary

KAG-2905

Checklist

Full changelog

  • [Implement ...]

Issue reference

Fix #[issue number]

@chronolaw chronolaw force-pushed the fix/add_lower_to_http_headers_expression branch from be961a6 to 7ac1f56 Compare November 2, 2023 07:38
@chronolaw chronolaw force-pushed the fix/add_lower_to_http_headers_expression branch from 7ac1f56 to e3f39ef Compare November 3, 2023 01:46
@pull-request-size pull-request-size bot added size/M and removed size/S labels Nov 3, 2023
@chronolaw chronolaw changed the title fix(router): add lower() to http headers expression fix(router): http headers sensitive for expressions and insensitive for traditional_compatible Nov 3, 2023
@pull-request-size pull-request-size bot added size/L and removed size/M labels Nov 3, 2023
@chronolaw chronolaw force-pushed the fix/add_lower_to_http_headers_expression branch 2 times, most recently from 0aef586 to 5a877b8 Compare November 3, 2023 05:06
@chronolaw chronolaw marked this pull request as ready for review November 3, 2023 05:07
kong/router/atc.lua Outdated Show resolved Hide resolved
@dndx dndx force-pushed the fix/add_lower_to_http_headers_expression branch from 8ca0937 to 443ee6c Compare November 7, 2023 09:15
@dndx dndx force-pushed the fix/add_lower_to_http_headers_expression branch from 9de7ff8 to b67244d Compare November 7, 2023 09:49
@dndx dndx force-pushed the fix/add_lower_to_http_headers_expression branch from 4350299 to 47b2e8d Compare November 8, 2023 06:08
@dndx dndx merged commit 6ce1262 into master Nov 8, 2023
@dndx dndx deleted the fix/add_lower_to_http_headers_expression branch November 8, 2023 06:36
chronolaw added a commit that referenced this pull request Jan 23, 2024
…pressions` flavor (#11905)

`traditional_compatible` flavor remains case insensitive to stay compatible with `traditional` flavor.

This change allow `expressions` route authors to pick whether they want case sensitive or insensitive matches.

KAG-2905

---------

Co-authored-by: Datong Sun <datong.sun@konghq.com>
chronolaw added a commit that referenced this pull request Jan 24, 2024
…pressions` flavor (#11905)

`traditional_compatible` flavor remains case insensitive to stay compatible with `traditional` flavor.

This change allow `expressions` route authors to pick whether they want case sensitive or insensitive matches.

KAG-2905

---------

Co-authored-by: Datong Sun <datong.sun@konghq.com>
chronolaw added a commit that referenced this pull request Feb 26, 2024
…pressions` flavor (#11905)

`traditional_compatible` flavor remains case insensitive to stay compatible with `traditional` flavor.

This change allow `expressions` route authors to pick whether they want case sensitive or insensitive matches.

KAG-2905

---------

Co-authored-by: Datong Sun <datong.sun@konghq.com>
chronolaw added a commit that referenced this pull request Feb 26, 2024
…pressions` flavor (#11905)

`traditional_compatible` flavor remains case insensitive to stay compatible with `traditional` flavor.

This change allow `expressions` route authors to pick whether they want case sensitive or insensitive matches.

KAG-2905

---------

Co-authored-by: Datong Sun <datong.sun@konghq.com>
chronolaw added a commit that referenced this pull request Mar 4, 2024
…pressions` flavor (#11905)

`traditional_compatible` flavor remains case insensitive to stay compatible with `traditional` flavor.

This change allow `expressions` route authors to pick whether they want case sensitive or insensitive matches.

KAG-2905

---------

Co-authored-by: Datong Sun <datong.sun@konghq.com>
locao pushed a commit that referenced this pull request Mar 5, 2024
…pressions` flavor (#11905)

`traditional_compatible` flavor remains case insensitive to stay compatible with `traditional` flavor.

This change allow `expressions` route authors to pick whether they want case sensitive or insensitive matches.

KAG-2905

---------

Co-authored-by: Datong Sun <datong.sun@konghq.com>
ADD-SP pushed a commit that referenced this pull request Mar 7, 2024
…pressions` flavor (#11905)

`traditional_compatible` flavor remains case insensitive to stay compatible with `traditional` flavor.

This change allow `expressions` route authors to pick whether they want case sensitive or insensitive matches.

KAG-2905

---------

Co-authored-by: Datong Sun <datong.sun@konghq.com>
ADD-SP pushed a commit that referenced this pull request Mar 7, 2024
…pressions` flavor (#11905)

`traditional_compatible` flavor remains case insensitive to stay compatible with `traditional` flavor.

This change allow `expressions` route authors to pick whether they want case sensitive or insensitive matches.

KAG-2905

---------

Co-authored-by: Datong Sun <datong.sun@konghq.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants