diff options
| author | mojganii <mojgan.jelodar@codic.se> | 2025-03-20 15:54:50 +0100 |
|---|---|---|
| committer | mojganii <mojgan.jelodar@codic.se> | 2025-04-15 15:38:45 +0200 |
| commit | f8aeeb42fa5cc7f1cc428bb4b34cb8d5de09bb51 (patch) | |
| tree | 7c6c740793dc53b39ae916d5f688a81e94218c6f /ios | |
| parent | 59f4f47a1ed3e89d6273ba1cb8a9749ef45e49b9 (diff) | |
| download | mullvadvpn-f8aeeb42fa5cc7f1cc428bb4b34cb8d5de09bb51.tar.xz mullvadvpn-f8aeeb42fa5cc7f1cc428bb4b34cb8d5de09bb51.zip | |
Call sendProblemReport from Rust in Swift
Diffstat (limited to 'ios')
7 files changed, 69 insertions, 6 deletions
diff --git a/ios/MullvadREST/ApiHandlers/RESTAPIProxy.swift b/ios/MullvadREST/ApiHandlers/RESTAPIProxy.swift index d45745446e..39585a88b7 100644 --- a/ios/MullvadREST/ApiHandlers/RESTAPIProxy.swift +++ b/ios/MullvadREST/ApiHandlers/RESTAPIProxy.swift @@ -263,6 +263,20 @@ extension REST { let newExpiry: Date } + public struct ProblemReportRequest: Codable, Sendable { + public let address: String + public let message: String + public let log: String + public let metadata: [String: String] + + public init(address: String, message: String, log: String, metadata: [String: String]) { + self.address = address + self.message = message + self.log = log + self.metadata = metadata + } + } + private struct SubmitVoucherRequest: Encodable, Sendable { let voucherCode: String } diff --git a/ios/MullvadREST/Extensions/String+UnsafePointer.swift b/ios/MullvadREST/Extensions/String+UnsafePointer.swift new file mode 100644 index 0000000000..4d60a1c77e --- /dev/null +++ b/ios/MullvadREST/Extensions/String+UnsafePointer.swift @@ -0,0 +1,16 @@ +// +// String+UnsafePointer.swift +// MullvadVPN +// +// Created by Mojgan on 2025-03-19. +// Copyright © 2025 Mullvad VPN AB. All rights reserved. +// +import Foundation +extension String { + func toUnsafePointer() -> UnsafePointer<UInt8>? { + guard let data = self.data(using: .utf8) else { return nil } + let buffer = UnsafeMutablePointer<UInt8>.allocate(capacity: data.count) + data.copyBytes(to: buffer, count: data.count) + return UnsafePointer(buffer) + } +} diff --git a/ios/MullvadREST/MullvadAPI/APIRequest/APIRequest.swift b/ios/MullvadREST/MullvadAPI/APIRequest/APIRequest.swift index 68e7e41663..7810230464 100644 --- a/ios/MullvadREST/MullvadAPI/APIRequest/APIRequest.swift +++ b/ios/MullvadREST/MullvadAPI/APIRequest/APIRequest.swift @@ -9,6 +9,8 @@ public enum APIRequest: Codable, Sendable { case getAddressList(_ retryStrategy: REST.RetryStrategy) case getRelayList(_ retryStrategy: REST.RetryStrategy, etag: String?) + case sendProblemReport(_ retryStrategy: REST.RetryStrategy, problemReportRequest: REST.ProblemReportRequest) + case createAccount(_ retryStrategy: REST.RetryStrategy) case getAccount(_ retryStrategy: REST.RetryStrategy, accountNumber: String) case deleteAccount(_ retryStrategy: REST.RetryStrategy, accountNumber: String) @@ -19,6 +21,8 @@ public enum APIRequest: Codable, Sendable { "get-address-list" case .getRelayList: "get-relay-list" + case .sendProblemReport: + "send-problem-report" case .createAccount: "create-account" case .getAccount: @@ -33,6 +37,7 @@ public enum APIRequest: Codable, Sendable { case let .getAddressList(strategy), let .getRelayList(strategy, _), + let .sendProblemReport(strategy, _), let .createAccount(strategy), let .getAccount(strategy, _), let .deleteAccount(strategy, _): diff --git a/ios/MullvadREST/MullvadAPI/MullvadApiRequestFactory.swift b/ios/MullvadREST/MullvadAPI/MullvadApiRequestFactory.swift index 298d3ac31f..970f70ef76 100644 --- a/ios/MullvadREST/MullvadAPI/MullvadApiRequestFactory.swift +++ b/ios/MullvadREST/MullvadAPI/MullvadApiRequestFactory.swift @@ -39,6 +39,13 @@ public struct MullvadApiRequestFactory: Sendable { retryStrategy.toRustStrategy(), etag )) + case let .sendProblemReport(retryStrategy, problemReportRequest): + MullvadApiCancellable(handle: mullvad_api_send_problem_report( + apiContext.context, + rawCompletionPointer, + retryStrategy.toRustStrategy(), + problemReportRequest.toRust() + )) case let .getAccount(retryStrategy, accountNumber: accountNumber): MullvadApiCancellable(handle: mullvad_api_get_account( apiContext.context, @@ -67,3 +74,24 @@ public struct MullvadApiRequestFactory: Sendable { extension REST { public typealias MullvadApiRequestHandler = (((MullvadApiResponse) throws -> Void)?) -> MullvadApiCancellable } + +private extension REST.ProblemReportRequest { + func toRust() -> UnsafePointer<SwiftProblemReportRequest> { + let structPointer = UnsafeMutablePointer<SwiftProblemReportRequest>.allocate(capacity: 1) + + let addressPointer = address.toUnsafePointer() + let messagePointer = message.toUnsafePointer() + let logPointer = log.toUnsafePointer() + + structPointer.initialize(to: SwiftProblemReportRequest( + address: addressPointer, + address_len: UInt(address.utf8.count), + message: messagePointer, + message_len: UInt(message.utf8.count), + log: logPointer, + log_len: UInt(log.utf8.count) + )) + + return UnsafePointer(structPointer) + } +} diff --git a/ios/MullvadREST/Transport/APITransport.swift b/ios/MullvadREST/Transport/APITransport.swift index 9af1f2779f..d0bf7db48d 100644 --- a/ios/MullvadREST/Transport/APITransport.swift +++ b/ios/MullvadREST/Transport/APITransport.swift @@ -34,7 +34,6 @@ public final class APITransport: APITransportProtocol { let apiRequest = requestFactory.makeRequest(request) return apiRequest { response in - let error: APIError? = if !response.success { APIError( statusCode: Int(response.statusCode), diff --git a/ios/MullvadVPN.xcodeproj/project.pbxproj b/ios/MullvadVPN.xcodeproj/project.pbxproj index 050bd50e1c..8c64f72340 100644 --- a/ios/MullvadVPN.xcodeproj/project.pbxproj +++ b/ios/MullvadVPN.xcodeproj/project.pbxproj @@ -977,6 +977,7 @@ F062000A2CB7EB42002E6DB9 /* CGSize+Helpers.swift in Sources */ = {isa = PBXBuildFile; fileRef = F06200092CB7EB42002E6DB9 /* CGSize+Helpers.swift */; }; F062000C2CB7EB5D002E6DB9 /* UIImage+Helpers.swift in Sources */ = {isa = PBXBuildFile; fileRef = F062000B2CB7EB5D002E6DB9 /* UIImage+Helpers.swift */; }; F062B94D2C16E09700B6D47A /* TunnelSettingsManagerTests.swift in Sources */ = {isa = PBXBuildFile; fileRef = F062B94C2C16E09700B6D47A /* TunnelSettingsManagerTests.swift */; }; + F0647C322D8AF1E800D2C489 /* String+UnsafePointer.swift in Sources */ = {isa = PBXBuildFile; fileRef = F0647C312D8AF1CD00D2C489 /* String+UnsafePointer.swift */; }; F072D3CF2C07122400906F64 /* SettingsUpdaterTests.swift in Sources */ = {isa = PBXBuildFile; fileRef = F072D3CE2C07122400906F64 /* SettingsUpdaterTests.swift */; }; F072D3D22C071AD100906F64 /* ShadowsocksLoaderTests.swift in Sources */ = {isa = PBXBuildFile; fileRef = F072D3D12C071AD100906F64 /* ShadowsocksLoaderTests.swift */; }; F073FCB32C6617D70062EA1D /* TunnelStore+Stubs.swift in Sources */ = {isa = PBXBuildFile; fileRef = F073FCB22C6617D70062EA1D /* TunnelStore+Stubs.swift */; }; @@ -2404,6 +2405,7 @@ F06200092CB7EB42002E6DB9 /* CGSize+Helpers.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = "CGSize+Helpers.swift"; sourceTree = "<group>"; }; F062000B2CB7EB5D002E6DB9 /* UIImage+Helpers.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = "UIImage+Helpers.swift"; sourceTree = "<group>"; }; F062B94C2C16E09700B6D47A /* TunnelSettingsManagerTests.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = TunnelSettingsManagerTests.swift; sourceTree = "<group>"; }; + F0647C312D8AF1CD00D2C489 /* String+UnsafePointer.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = "String+UnsafePointer.swift"; sourceTree = "<group>"; }; F072D3CE2C07122400906F64 /* SettingsUpdaterTests.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = SettingsUpdaterTests.swift; sourceTree = "<group>"; }; F072D3D12C071AD100906F64 /* ShadowsocksLoaderTests.swift */ = {isa = PBXFileReference; lastKnownFileType = sourcecode.swift; path = ShadowsocksLoaderTests.swift; sourceTree = "<group>"; }; F073FCB22C6617D70062EA1D /* TunnelStore+Stubs.swift */ = {isa = PBXFileReference; fileEncoding = 4; lastKnownFileType = sourcecode.swift; path = "TunnelStore+Stubs.swift"; sourceTree = "<group>"; }; @@ -4325,6 +4327,7 @@ 7AD63A422CDA661B00445268 /* Extensions */ = { isa = PBXGroup; children = ( + F0647C312D8AF1CD00D2C489 /* String+UnsafePointer.swift */, 7AD63A432CDA662900445268 /* UInt+Counting.swift */, ); path = Extensions; @@ -5812,6 +5815,7 @@ F06045EA2B23217E00B2D37A /* ShadowsocksTransport.swift in Sources */, 06799AFC28F98EE300ACD94E /* AddressCache.swift in Sources */, 7ADCB2D82B6A6EB300C88F89 /* AnyRelay.swift in Sources */, + F0647C322D8AF1E800D2C489 /* String+UnsafePointer.swift in Sources */, 06799AF028F98E4800ACD94E /* REST.swift in Sources */, 06799ADF28F98E4800ACD94E /* RESTDevicesProxy.swift in Sources */, 06799ADA28F98E4800ACD94E /* RESTResponseHandler.swift in Sources */, diff --git a/ios/MullvadVPN/View controllers/ProblemReport/ProblemReportInteractor.swift b/ios/MullvadVPN/View controllers/ProblemReport/ProblemReportInteractor.swift index 6640473e4a..55fc1df7d3 100644 --- a/ios/MullvadVPN/View controllers/ProblemReport/ProblemReportInteractor.swift +++ b/ios/MullvadVPN/View controllers/ProblemReport/ProblemReportInteractor.swift @@ -80,13 +80,10 @@ final class ProblemReportInteractor: @unchecked Sendable { metadata: metadataDict ) - _ = self.apiProxy.sendProblemReport( - request, - retryStrategy: .default - ) { result in + _ = self.apiProxy.mullvadSendProblemReport(request, retryStrategy: .default, completionHandler: { result in DispatchQueue.main.async { completion(result) } - } + }) } } |
