fix: custom host with a path

[ioannisj]

💡 Motivation and Context

Closes #274

💚 How did you test it?

Added some unit tests

📝 Checklist

• I reviewed the submitted code.
• I added tests to verify the changes.
• I updated the docs if needed.
• No breaking change or entry added to the changelog.

[pauldambra]

lots of tests 👍

[marandaneto]

left a suggestion but overall looks gooood

[bcheidemann]

@ioannisj guessing I can close bcheidemann-kmbal#1 as this PR does (near enough) the same thing?

EDIT: I see I raised the PR against a fork anyway 🙈

[ioannisj]

@ioannisj guessing I can close bcheidemann-kmbal#1 as this PR does (near enough) the same thing?

EDIT: I see I raised the PR against a fork anyway 🙈

Hey @bcheidemann, so sorry! I completely missed that and didn't see a notification! That's a pity, cause it was looks like most (if not all) of the work was already done for this!

Yeah, I think that the fact that it was not against this repo had something to do with it! Thank you for the effort and time though! ❤️

[bcheidemann]

@ioannisj guessing I can close bcheidemann-kmbal#1 as this PR does (near enough) the same thing?

EDIT: I see I raised the PR against a fork anyway 🙈

Hey @bcheidemann, so sorry! I completely missed that and didn't see a notification! That's a pity, cause it was looks like most (if not all) of the work was already done for this!

Yeah, I think that the fact that it was not against this repo had something to do with it! Thank you for the effort and time though! ❤️

Hey, no problem! Very much my fault for not raising the PR against the right repo. TBH I'm glad to have another implementation of the same fix; reviewing it was a good learning opportunity for me 😉