-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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-7202 missing respond file overload #4114
KTOR-7202 missing respond file overload #4114
Conversation
Hey @jacekgajek, could you please change the target branch to |
c7ce492
to
91ee297
Compare
91ee297
to
ddbf476
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jacekgajek, thanks for the PR!
@osipxd, can you please take a quick look? The PR also contains my changes, and I want them to be reviewed too |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good job! Just a couple of stylistic comments
ktor-server/ktor-server-core/jvm/src/io/ktor/server/response/ApplicationResponseFunctionsJvm.kt
Outdated
Show resolved
Hide resolved
ktor-server/ktor-server-core/jvm/src/io/ktor/server/response/ApplicationResponseFunctionsJvm.kt
Outdated
Show resolved
Hide resolved
ktor-server/ktor-server-core/jvm/src/io/ktor/server/response/ApplicationResponseFunctionsJvm.kt
Outdated
Show resolved
Hide resolved
ktor-server/ktor-server-core/jvm/src/io/ktor/server/response/ApplicationResponseFunctionsJvm.kt
Outdated
Show resolved
Hide resolved
ktor-server/ktor-server-core/jvm/src/io/ktor/server/response/ApplicationResponseFunctionsJvm.kt
Outdated
Show resolved
Hide resolved
ktor-server/ktor-server-core/jvm/src/io/ktor/server/response/ApplicationResponseFunctionsJvm.kt
Outdated
Show resolved
Hide resolved
…pplicationResponseFunctionsJvm.kt Co-authored-by: Osip Fatkullin <osip.fatkullin@gmail.com>
…-respondFile-overload' into fork/jacekgajek/jacekgajek/KTOR-7202-Missing-respondFile-overload
Subsystem
ktor-server-core
Motivation
Resolves https://youtrack.jetbrains.com/issue/KTOR-7202/Missing-respondFile-overload
Solution
Add missing overload