Use host:port instead of URL and TLS test fixes

[cretz]

What was changed

• Accept target_host now that is just host:port instead of full URL
• Update Temporalite and fix TLS tests

Checklist

1. Closes Accept host:port for client instead of only URL #79

[bergundy]

I'd consider accepting None here too

[cretz]

That seems ambiguous. We used to accept None here when this wasn't how you chose whether TLS is used, but now a tls=False is clearer. The parameter changed from "tls configuration" to "tls enabled, with configuration if necessary".

[bergundy]

Are you eventually going to remove the go SDK dependency from here?

[cretz]

Are you eventually going to remove the go SDK dependency from here?

Yes, but while I continue to have to build Temporalite which has a Go dependency too, I am in no hurry. If Temporalite binaries were released standalone I would remove Go completely from the test process. I have opened temporalio/temporalite-archived#94.

[cretz]

Hrmm, the upgrade of Temporalite to 1.17.x is causing an issue with start-then-immediate-query from temporalio/temporal#2826. I am talking with @yiminc and the server team. I may have to downgrade or alter code to ignore FailedPrecondition.

EDIT: Altered code to fix this