@Mark Gilchrist I also favor the idea of returning Result, I think that is at least the more correct way to handle the error and probably the nicer (in my opinion)

it is what drew me to fuel in the first place

ah ha same mind! hope you like Fuel though!

New version is release!

i seem to be running into the issue that a server redirects to urls that contain spaces in the url.. Fuel seems to barf on that

is there any documentation for how to implement a interceptor to fix that ?

.replace(" ", "%20")

should be good enough atm i set redirects to not be allowed and loop until there is no redirect anymore but i'd like to get rid of that ugly code

would you like to raise and issue or pr? https://github.com/kittinunf/Fuel/issues

i will, probably when i have some time to put a good issue together

@Nikky the latest version improves a lot regarding with redirect. Can you try it?

v1.15.0 to be exact

i did try it.. makes it al ittle easier... not sure how i would use it to fix the invalid redirects.. but i am going to tinker with it more in a bit

actually URL will be passing into Encoder which will encode

" "

to

"%20"

, maybe I am not sure what exactly is the problem

can you try to write and issue in the repo? even better writing a test to demonstrate the problem will be even better

on fuel .15.0 it seems to me like the .header() call is not working using

.apply { httpHeaders["Content-Type"] = "application/json" }

works though

it seems to be deprecated 😕

okay well i figured it out.. it seems like headers() doesn ot override the headers anymore and there is a default content type already which it gets appended to

@Nikky you can do

header(mapOf("Content-Type" to "application/protobuf"))

and it should override

i will try that out.. thanks for the suggestion

also, you can use

r.headers

like

r.headers["foo"] = "bar"

too

httpHeaders

is deprecated

any chance of getting fuel-coroutines for 1.3-M1 ?

i cannot mix experimental coroutines with the new ones

i just ended up copying fuel-coroutines into a subproject and adapted the buildscript, removed experimental from the imports and it all works

I would do a pr for this if @kittinunf thinks this is a good idea? or if you want to do it @Nikky?

I would not know the procedures.. But i'd rather use a somewhat official version of the package.. Instead of a local copy

here is what i did though: https://github.com/elytra/Voodoo/tree/1369ca6cbef7049fce4f14549955b668a96fad17/fuel-coroutines

since its essentially just one file it was easy enough to adapt.. until it supported

also 1.3 has this https://blog.jetbrains.com/kotlin/2018/07/see-whats-coming-in-kotlin-1-3-m1/#successorfailure

seems awfully close to Result