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

strip_path explicit support, and x-kong-route-defaults support #3273

Merged
merged 4 commits into from Apr 26, 2021

Conversation

dimitropoulos
Copy link
Contributor

@dimitropoulos dimitropoulos commented Apr 9, 2021

closes: INS-122, closes INS-555, closes INS-554

@develohpanda develohpanda added the PA-openapi-2-kong Package: OpenAPI 2 Kong label Apr 19, 2021
Tieske added a commit that referenced this pull request Apr 24, 2021
This allows to set the properties that cannot be derived from the
OAS spec itself, like preserve_host, etc.

The precedence is most-specific to least-specific, so defaults
at operations level override path level, which in turn overrides
root level.

replaces #3273
Tieske added a commit that referenced this pull request Apr 24, 2021
This allows to set the properties that cannot be derived from the
OAS spec itself, like preserve_host, etc.

The precedence is most-specific to least-specific, so defaults
at operations level override path level, which in turn overrides
root level.

replaces #3273
Copy link
Member

@Tieske Tieske left a comment

Choose a reason for hiding this comment

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

I think we should close this and replace it with #3319

supports selecting but not merging defaults
Co-authored-by: Thijs Schreijer <thijs@thijsschreijer.nl>
@dimitropoulos dimitropoulos merged commit cde2516 into Kong:develop Apr 26, 2021
@dimitropoulos dimitropoulos deleted the feature/ins-122_stripPath branch April 26, 2021 16:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PA-openapi-2-kong Package: OpenAPI 2 Kong
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants