Hey @k8mil, can you check if it works as expected when you change your baseURL to be just https://google.com and path to return /123/somepath?X-ABC-Asd=123?

@pedrovereza
Thanks for your response.

I've created a sample repo with the test code. Just clone and run it.

https://github.com/k8mil/MoyaBaseURL

Below you can see, screen from Charles with 3 options:

First

request was performed with Alamofire using
URL(string: "https://google.com/123/somepath?X-ABC-Asd=123")!

Second

request was performed using MoyaProvider with
baseURL = "https://google.com/123/somepath?X-ABC-Asd=123"

Third

request was performed using MoyaProvider with
baseURL = "https://google.com and path = "/123/somepath?X-ABC-Asd=123"

And only Alamofire produces that what I want to achieve :)

Important thing is that I'm using Moya in my project and the consuming my backend API with Moya is really great and works perfectly, but I have some case that I have to use the URL(which is not related with backedAPI) that I received from some service and I shouldn't modify that URL. Unfortunately, as you can see in the Second and Thrid example the URL is modified by removing/changing the ? char.

Hey @k8mil thanks for the detailed explanation and examples 👍

I think I have a fix in #1053, care to take a look? 😉

Very interesting use case. Thanks for implementing this @pedrovereza 😄

Thanks @pedrovereza !
I've tested it and it works fine.
So, now I wait till it will be merged with master 👍 :)

Modify the source code in file URL+Moya.swift as below:

public extension URL {

    /// Initialize URL from Moya's `TargetType`.
    init<T: TargetType>(target: T) {
        // When a TargetType's path is empty, URL.appendingPathComponent may introduce trailing /, which may not be wanted in some cases
        // See: https://github.com/Moya/Moya/pull/1053
        // And: https://github.com/Moya/Moya/issues/1049
        let targetPath = target.path
        if targetPath.isEmpty {
            self = target.baseURL
        } else {
            let tag = target.baseURL.absoluteString.hasSuffix("/") ? "" : "/"
            self = .init(string: target.baseURL.absoluteString + tag + targetPath)!
        }
    }
}