Hello, question about HttpsRedirect feature. I just install it and all my tests are failed now becau...

# ktor

[vincent.brule]

Hello, question about HttpsRedirect feature. I just install it and all my tests are failed now because handleRequest call are made with http so the request is redirected. Is it possible to have directly https call in handleRequest please ?

[Deactivated User]

I’m not sure if I understand what’s going on do you have a small snippet with the feature install + route + test that I can try?

[vincent.brule]

I don’t have a small snippet but I can show you the problem without I think

[vincent.brule]

To install the feature I just put install(HttpsRedirect)

[Deactivated User]

aha

[vincent.brule]

I have a route for authentication /authenticate

[vincent.brule]

And a small part of the test is handleRequest(HttpMethod.Get, “/authenticate”) {
addHeader(“Host”, “127.0.0.1”)
}.let { call ->
val location = call.response.headers[“Location”] ?: “”
assertEquals(
“https://accounts.google.com/o/oauth2/auth?client_id=$configClientID&redirect_uri=http%3A%2F%2F127.0.0.1%2Fauthenticate&scope=email&state=****&response_type=code“,
location.maskState()
)
val stateInfo = Regex(“state=(\\w+)“).find(location)
state = stateInfo!!.groupValues[1]
}

[vincent.brule]

but the assert failed because I have https://127.0.0.1/authenticate instead of google address

[Deactivated User]

aha

[Deactivated User]

okay, let me check, I think you was able to specify to handleRequest with the https protocol, but have to check how

[Deactivated User]

and it seems that in the obvious pages of the documentation:
• https://ktor.io/servers/testing.html
• https://ktor.io/features/https-redirect.html
nothing is said, so I will update the documentation too

[Deactivated User]

handleRequest(HttpMethod.Get, "/", {
    addHeader(HttpHeaders.XForwardedProto, "https")
}).let { call ->
    assertEquals(HttpStatusCode.OK, call.response.status())
}

[Deactivated User]

can you try this?

[Deactivated User]

addHeader(HttpHeaders.XForwardedProto, "https")

[Deactivated User]

let me know if it works, and i’ll update the documentation


I grabbed that sample from here: https://github.com/ktorio/ktor/blob/bb0765ce00e5746c954fea70270cf7d802a40648/ktor-server/ktor-server-tests/test/io/ktor/tests/server/features/HttpsRedirectFeatureTest.kt#L31-L49

[Deactivated User]

you will probably need to add application.install(XForwardedHeadersSupport) too for it to work I guess. But you can add it only to the tests without affecting your main app

[Deactivated User]

the other thing I can think of, is to just not install the HttpsRedirect when doing the tests. For example by adding a installHttpsRedirect: Boolean to the function that installs it and conditionally install it based on that flag

[Deactivated User]

I guess you can also use the uninstall method: application.uninstall(HttpsRedirect)

[vincent.brule]

Sorry I was not here, I will try right now, thanks for your help

[vincent.brule]

So this addHeader(HttpHeaders.XForwardedProto, “https”)
didn’t work

[Deactivated User]

I have documented the proposed solutions here: https://github.com/ktorio/ktorio.github.io/commit/038be56a327709436c0a65006746fd3181d95ecc

[Deactivated User]

addHeader(HttpHeaders.XForwardedProto, "https") with or without adding the application.install(XForwardedHeadersSupport)?

[vincent.brule]

Sorry I forgot to install it

[vincent.brule]

So ok that’s working but can you develop how can I put a flag to just don’t install this feature because it’s will be really more convenient. With the other method, I have to change each test …

[Deactivated User]

http://ktor.io/features/https-redirect.html#do-not-install-the-feature-when-testing-or-uninstall-it

[vincent.brule]

Ok perfect, problem solved ! Thanks 🙂

[Deactivated User]

nice 🙂

[vincent.brule]

Just to inform you, I won’t use application.uninstall(HttpsRedirect) but in my case that didn’t work with this line and the feature keep be installed

[Deactivated User]

Uhm. That might be a bug then. I’ll investigate. Thanks for informing 👍