Renamed some variables to make it a bit clearer and made sure it supported a port in the url

pull/592/head
Morgan Pretty 4 years ago
parent dad264c239
commit ea7603bf3b

@ -97,23 +97,25 @@ public final class OpenGroupManager: NSObject {
public func hasExistingOpenGroup(roomToken: String, server: String, publicKey: String, using transaction: YapDatabaseReadWriteTransaction, dependencies: OGMDependencies = OGMDependencies()) -> Bool { public func hasExistingOpenGroup(roomToken: String, server: String, publicKey: String, using transaction: YapDatabaseReadWriteTransaction, dependencies: OGMDependencies = OGMDependencies()) -> Bool {
guard let serverUrl: URL = URL(string: server) else { return false } guard let serverUrl: URL = URL(string: server) else { return false }
let schemeFreeServer: String = (serverUrl.host ?? server) let serverHost: String = (serverUrl.host ?? server)
let schemeFreeDefaultServer: String = OpenGroupAPI.defaultServer.substring(from: "http://".count) let serverPort: String = (serverUrl.port.map { ":\($0)" } ?? "")
let defaultServerHost: String = OpenGroupAPI.defaultServer.substring(from: "http://".count)
var serverOptions: Set<String> = Set([ var serverOptions: Set<String> = Set([
schemeFreeServer, server,
"http://\(schemeFreeServer)", "\(serverHost)\(serverPort)",
"https://\(schemeFreeServer)" "http://\(serverHost)\(serverPort)",
"https://\(serverHost)\(serverPort)"
]) ])
if schemeFreeServer == OpenGroupAPI.legacyDefaultServerDNS { if serverHost == OpenGroupAPI.legacyDefaultServerDNS {
let defaultServerOptions: Set<String> = Set([ let defaultServerOptions: Set<String> = Set([
schemeFreeDefaultServer, defaultServerHost,
OpenGroupAPI.defaultServer, OpenGroupAPI.defaultServer,
"https://\(schemeFreeDefaultServer)" "https://\(defaultServerHost)"
]) ])
serverOptions = serverOptions.union(defaultServerOptions) serverOptions = serverOptions.union(defaultServerOptions)
} }
else if schemeFreeServer == schemeFreeDefaultServer { else if serverHost == defaultServerHost {
let legacyServerOptions: Set<String> = Set([ let legacyServerOptions: Set<String> = Set([
OpenGroupAPI.legacyDefaultServerDNS, OpenGroupAPI.legacyDefaultServerDNS,
"http://\(OpenGroupAPI.legacyDefaultServerDNS)", "http://\(OpenGroupAPI.legacyDefaultServerDNS)",

Loading…
Cancel
Save