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

Full support for special characters in asterisk paths #1569

Merged
merged 2 commits into from
Jan 2, 2025

Conversation

barchw
Copy link
Contributor

@barchw barchw commented Dec 27, 2024

Description

Changes proposed in this pull request:

  • Full support for special characters in asterisk paths
  • Fix support for space in url (%20 and +)

Pre-Merge Checklist

  • As a PR reviewer, verify code coverage and evaluate if it is acceptable.

Related issues

@barchw barchw requested a review from a team as a code owner December 27, 2024 12:48
@kyma-bot kyma-bot added cla: yes Indicates the PR's author has signed the CLA. size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Dec 27, 2024
@kyma-bot kyma-bot added the lgtm Looks good to me! label Dec 31, 2024
@barchw barchw merged commit 91747c5 into kyma-project:main Jan 2, 2025
15 checks passed
@barchw barchw deleted the special branch January 2, 2025 07:23
mluk-sap added a commit to mluk-sap/api-gateway that referenced this pull request Jan 2, 2025
* 'main' of https://github.com/kyma-project/api-gateway:
  Add information on local domain to the quick start guide (kyma-project#1579)
  fix a broken link (kyma-project#1576)
  Full support for special characters in asterisk paths (kyma-project#1569)
  Update API Gateway tutorials (kyma-project#1559)
  Update cicd docs (kyma-project#1573)
  Adjust pull request cicd workflow to the new strategy (kyma-project#1571)
Ressetkk pushed a commit to Ressetkk/api-gateway that referenced this pull request Jan 3, 2025
)

* Full support for special characters in asterisk paths

* Fix regex pattern
Ressetkk pushed a commit to Ressetkk/api-gateway that referenced this pull request Jan 3, 2025
)

* Full support for special characters in asterisk paths

* Fix regex pattern
@Ressetkk Ressetkk mentioned this pull request Jan 3, 2025
Ressetkk added a commit that referenced this pull request Jan 7, 2025
* Faster requeue when object has been modified error happens (#1496)

* Faster requeue when object has been modified error happens

* Add api-gateway metrics

* Register metrics

* Add unit test

* Fix lint

* chore: prepare 2.10.2

* QOL updates to APIRule v2alpha1 integration tests (#1507)

* Remove dupplicated inits from apirule tests

* Add template file name

* switch files

* Scale down oathkeeper for faster startup

* Revert delay

* Conditional teardown

* Re-add len parameter to RNG

* Adapt according to review comments

* Full support for special characters in asterisk paths (#1569)

* Full support for special characters in asterisk paths

* Fix regex pattern

* Only use scenario initializer for v2alpha1 tests (#1521)

* Apply suggestions from code review

Co-authored-by: Natalia Sitko <80401180+nataliasitko@users.noreply.github.com>

---------

Co-authored-by: Patryk Strugacz <werdes72@users.noreply.github.com>
Co-authored-by: Bartosz Chwila <103247439+barchw@users.noreply.github.com>
Co-authored-by: Natalia Sitko <80401180+nataliasitko@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes Indicates the PR's author has signed the CLA. lgtm Looks good to me! size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants