tree b9c8d6664464dc805bbcc9cbfe0ca2a429463428
parent 32c990a1a7f73062160b22d144599dca5fffd650
author Nate Bosch <nbosch@google.com> 1638563977 -0800
committer GitHub <noreply@github.com> 1638563977 -0800
gpgsig -----BEGIN PGP SIGNATURE-----
 
 wsBcBAABCAAQBQJhqoCJCRBK7hj4Ov3rIwAA/gkIAGwcq4LK6V0KBLSl3UHngFhQ
 plgEFrUD2zBjiBNPNgPKauzKo7EjMsxk32XSaAOg4Xxhzl45da0L9PUm1FfSN0nu
 jkumbaf8yIz5I2RLq9j9HjtcBFhCiU/RbiC6mPA7blbLHRZsBdElUkqQpg5KDE87
 6L3A/MhHUvdgylP5HX5o+Xw/ZZ6c2RAKs/A+ux5G8IfVYQRPZ/idEWschjGFs2Gi
 Eo30eNzT6cMC7DuKB7BTaXNr0kUiw225H0aGIAODfcfuS39EZwX5VPH+P4jIOzJS
 BswYoERuzJ3HyeCwl/0nBoTpErHivueqxCYWkGOe/M/WisZzgA33+JLu1h0rkw0=
 =rEfj
 -----END PGP SIGNATURE-----
 

Use URI class over manual string manipulation (#1641)

Avoid building URIs manually with brittle strings. This also makes it
more clear that the Observatory URI is embedding a path and query string
into the fragment, instead of letting it look like part of the URI
itself.

Pull out `serverUri` as a separate variable and use `!` immediately. If
a null shows up this would be a more clear failure than trying to later
treat the string`'null'` as a URI.