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

KTOR-7543 Fix a couple backwards compatibility issues in 3.0 #4377

Merged
merged 2 commits into from
Oct 8, 2024

Conversation

bjhham
Copy link
Contributor

@bjhham bjhham commented Oct 8, 2024

Subsystem
Server, Routing

Motivation
KTOR-7543 Common backwards compatibility problems for 3.0

These are a couple of items that are tough to solve with structural replacement, and I've found are very common in projects.

Solution

  • Added coroutine scope to RoutingContext so that launch references don't break.
  • Added property alias through an extension for call, which is generally always used for interceptors - this only applies to pipeline contexts where the context extends ApplicationCall

@bjhham bjhham requested a review from e5l October 8, 2024 08:00
@e5l
Copy link
Member

e5l commented Oct 8, 2024

@bjhham, please check API dump before merging

@bjhham bjhham merged commit 9f663af into main Oct 8, 2024
6 of 14 checks passed
@bjhham bjhham deleted the bjhham/pipeline-backwards-compat branch October 8, 2024 09:40
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.

2 participants