Remove normalization and prefer .build.

Author: ioquatix

lib/async/redis/endpoint.rb

@@ -36,7 +36,7 @@ def self.remote(host, port = 6379, **options)
 			}
 
 			def self.parse(string, endpoint = nil, **options)
-				url = URI.parse(string).normalize
+				url = URI.parse(string)
 
 				return self.new(url, endpoint, **options)
 			end
@@ -55,10 +55,14 @@ def self.for(scheme, host, credentials: nil, port: nil, database: nil, **options
 					path = "/#{database}"
 				end
 
-				host = host.include?(":") ? "[#{host}]" : host
-				
 				self.new(
-					uri_klass.new(scheme, credentials&.join(":"), host, port, nil, path, nil, nil, nil).normalize,
+					uri_klass.build(
+						scheme: scheme,
+						userinfo: credentials&.join(":"),
+						host: host,
+						port: port,
+						path: path,
+					),
 					**options
 				)
 			end

test/async/redis/endpoint.rb

@@ -81,15 +81,15 @@
 	with ".for" do
 		it "handles IPv4 addresses correctly" do
 			endpoint = Async::Redis::Endpoint.for("redis", "127.0.0.1", port: 6380)
-			expect(endpoint.url.to_s).to be == "redis://127.0.0.1:6380/"
+			expect(endpoint.url.to_s).to be == "redis://127.0.0.1:6380"
 			expect(endpoint.url.host).to be == "127.0.0.1"
 			expect(endpoint.url.hostname).to be == "127.0.0.1"
 			expect(endpoint.port).to be == 6380
 		end
 
 		it "handles IPv6 addresses correctly" do
 			endpoint = Async::Redis::Endpoint.for("redis", "::1", port: 6380)
-			expect(endpoint.url.to_s).to be == "redis://[::1]:6380/"
+			expect(endpoint.url.to_s).to be == "redis://[::1]:6380"
 			expect(endpoint.url.host).to be == "[::1]"
 			expect(endpoint.url.hostname).to be == "::1"
 			expect(endpoint.port).to be == 6380
@@ -98,15 +98,15 @@
 		it "handles expanded IPv6 addresses correctly" do
 			ipv6 = "2600:1f28:372:c404:5c2d:ce68:3620:cc4b"
 			endpoint = Async::Redis::Endpoint.for("redis", ipv6, port: 6380)
-			expect(endpoint.url.to_s).to be == "redis://[#{ipv6}]:6380/"
+			expect(endpoint.url.to_s).to be == "redis://[#{ipv6}]:6380"
 			expect(endpoint.url.host).to be == "[#{ipv6}]"
 			expect(endpoint.url.hostname).to be == ipv6
 			expect(endpoint.port).to be == 6380
 		end
 
 		it "handles credentials correctly" do
 			endpoint = Async::Redis::Endpoint.for("redis", "localhost", credentials: ["user", "pass"], port: 6380)
-			expect(endpoint.url.to_s).to be == "redis://user:pass@localhost:6380/"
+			expect(endpoint.url.to_s).to be == "redis://user:pass@localhost:6380"
 			expect(endpoint.url.userinfo).to be == "user:pass"
 			expect(endpoint.credentials).to be == ["user", "pass"]
 		end
@@ -120,7 +120,7 @@
 
 		it "handles secure connections correctly" do
 			endpoint = Async::Redis::Endpoint.for("rediss", "localhost")
-			expect(endpoint.url.to_s).to be == "rediss://localhost/"
+			expect(endpoint.url.to_s).to be == "rediss://localhost"
 			expect(endpoint).to be(:secure?)
 		end