< e5l> Hi can you provide some more detail on this I m facin kotlinlang #ktor

Channels

100daysofcode

100daysofkotlin

100daysofkotlin-2021

androidgithubprojects

cincinnati-user-group

competitive-programming

event21-community-content

java-to-kotlin-refactoring

jvm-ir-backend-feedback

kotlin-multiplatform-contest

kotlinultimatechallenge

myndocs-oauth2-server

naming

navigation-architecture-component

practical-functional-programming

proguard

prozis-android-backup

refactoring-to-kotlin

russian-kotlinasfirst

skia-wasm-interop-temp

stayhungrystayfoolish

swagger-gradle-codegen

tricity-kotlin-user-group

Title

yshrsmz

06/18/2019, 10:54 AM

@e5l Hi, can you provide some more detail on this? I'm facing this error, but unable to resolve for now.

install(HttpCookies) {
  storage = CustomStorage()
}

above code does not work, as

AcceptAllCookiesStorage

is provided as a default value for Config class. https://kotlinlang.slack.com/archives/C0A974TJ9/p1560522327059700?thread_ts=1560519079.059100&cid=C0A974TJ9

Copying entire HttpCookies file and edit Config's default value seems to solve the problem, but is this it?

e5l

06/18/2019, 11:49 AM

Hi @yshrsmz, how did you implement

CustomStorage

yshrsmz

06/18/2019, 11:52 AM

@e5l Hi, CustomStorage implementation is not problem, because app crashes here( https://github.com/ktorio/ktor/blob/1.2.1/ktor-client/ktor-client-core/common/src/io/ktor/client/features/cookies/HttpCookies.kt#L50 and https://github.com/ktorio/ktor/blob/1.2.1/ktor-client/ktor-client-core/common/src/io/ktor/client/features/cookies/HttpCookies.kt#L32)

e5l

06/18/2019, 11:54 AM

Ok, I’ve got it. Probably we should release the bug fix.

🙂 1

yshrsmz

06/18/2019, 11:57 AM

that would be great! 🙏

Andy Victors

06/25/2019, 12:22 PM

Is there an issue for this already? I'm also blocked with this problem

#ktor Join Slack