ConnectionString shouldn't use verify()

verify() is non-fatal on production builds but fatal on debug builds, which makes it very difficult to use correctly. We should replace the usages of verify() in ConnectionString with uassert(), at least until we can get rid of all assertions from ConnectionString and use Status returns instead.