Importing: Import from zip in a background thread
Signed-off-by: Roopesh Chander <roop@roopc.net>
This commit is contained in:
parent
203d6b4596
commit
b1ae13652c
|
@ -160,11 +160,8 @@ class TunnelsListTableViewController: UIViewController {
|
|||
func importFromFile(url: URL) {
|
||||
guard let tunnelsManager = tunnelsManager else { return }
|
||||
if (url.pathExtension == "zip") {
|
||||
let zipImporter = ZipImporter(url: url)
|
||||
let configs: [TunnelConfiguration?]
|
||||
do {
|
||||
configs = try zipImporter.importConfigFiles()
|
||||
} catch (let error) {
|
||||
ZipImporter.importConfigFiles(from: url) { (configs, error) in
|
||||
if let error = error {
|
||||
ErrorPresenter.showErrorAlert(error: error, from: self)
|
||||
return
|
||||
}
|
||||
|
@ -175,6 +172,7 @@ class TunnelsListTableViewController: UIViewController {
|
|||
self?.showErrorAlert(title: "Created \(numberSuccessful) tunnels",
|
||||
message: "Created \(numberSuccessful) of \(configs.count) tunnels from zip archive")
|
||||
}
|
||||
}
|
||||
} else /* if (url.pathExtension == "conf") -- we assume everything else is a conf */ {
|
||||
let fileBaseName = url.deletingPathExtension().lastPathComponent.trimmingCharacters(in: .whitespacesAndNewlines)
|
||||
if let fileContents = try? String(contentsOf: url),
|
||||
|
|
|
@ -8,13 +8,11 @@ enum ZipImporterError: Error {
|
|||
}
|
||||
|
||||
class ZipImporter {
|
||||
let url: URL
|
||||
init(url: URL) {
|
||||
self.url = url
|
||||
}
|
||||
|
||||
func importConfigFiles() throws -> [TunnelConfiguration?] {
|
||||
var unarchivedFiles: [(fileName: String, contents: Data)] = try ZipArchive.unarchive(url: url, requiredFileExtensions: ["conf"])
|
||||
static func importConfigFiles(from url: URL, completion: @escaping ([TunnelConfiguration?], Error?) -> Void) {
|
||||
DispatchQueue.global(qos: .userInitiated).async {
|
||||
var unarchivedFiles: [(fileName: String, contents: Data)]
|
||||
do {
|
||||
unarchivedFiles = try ZipArchive.unarchive(url: url, requiredFileExtensions: ["conf"])
|
||||
|
||||
for (i, unarchivedFile) in unarchivedFiles.enumerated().reversed() {
|
||||
let fileBaseName = URL(string: unarchivedFile.fileName)?.deletingPathExtension().lastPathComponent
|
||||
|
@ -28,6 +26,10 @@ class ZipImporter {
|
|||
if (unarchivedFiles.isEmpty) {
|
||||
throw ZipImporterError.noTunnelsInZipArchive
|
||||
}
|
||||
} catch (let error) {
|
||||
DispatchQueue.main.async { completion([], error) }
|
||||
return
|
||||
}
|
||||
|
||||
unarchivedFiles.sort { $0.fileName < $1.fileName }
|
||||
var configs = Array<TunnelConfiguration?>(repeating: nil, count: unarchivedFiles.count)
|
||||
|
@ -43,6 +45,7 @@ class ZipImporter {
|
|||
}
|
||||
configs[i] = tunnelConfig
|
||||
}
|
||||
return configs
|
||||
DispatchQueue.main.async { completion(configs, nil) }
|
||||
}
|
||||
}
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue