Merge branch 'main' into liveActivity

Author: cbaker6

.github/workflows/release.yml

@@ -4,7 +4,7 @@ on:
     types: [published]
 env:
   CI_XCODE_14: '/Applications/Xcode_14.3.1.app/Contents/Developer'
-  CI_XCODE_LATEST: '/Applications/Xcode_16.1.app/Contents/Developer'
+  CI_XCODE_LATEST: '/Applications/Xcode_16.2.app/Contents/Developer'
 
 jobs:
   cocoapods:

CHANGELOG.md

@@ -2,9 +2,16 @@
 # Parse-Swift Changelog
 
 ### main
-[Full Changelog](https://github.com/netreconlab/Parse-Swift/compare/5.11.3...main), [Documentation](https://swiftpackageindex.com/netreconlab/Parse-Swift/main/documentation/parseswift)
+[Full Changelog](https://github.com/netreconlab/Parse-Swift/compare/5.11.4...main), [Documentation](https://swiftpackageindex.com/netreconlab/Parse-Swift/main/documentation/parseswift)
 * _Contributing to this repo? Add info about your change here to be included in the next release_
 
+### 5.11.4
+[Full Changelog](https://github.com/netreconlab/Parse-Swift/compare/5.11.3...5.11.4), [Documentation](https://swiftpackageindex.com/netreconlab/Parse-Swift/5.11.4/documentation/parseswift)
+
+__Fixes__
+* Encode plus symbol in query parameter URI's to server ([#191](https://github.com/netreconlab/Parse-Swift/pull/191)), thanks to [Corey Baker](https://github.com/cbaker6).
+* Encode Firebase notification keys correctly ([#187](https://github.com/netreconlab/Parse-Swift/pull/187)), thanks to [Corey Baker](https://github.com/cbaker6).
+
 ### 5.11.3
 [Full Changelog](https://github.com/netreconlab/Parse-Swift/compare/5.11.2...5.11.3), [Documentation](https://swiftpackageindex.com/netreconlab/Parse-Swift/5.11.3/documentation/parseswift)
 

Sources/ParseSwift/API/API+Command.swift

@@ -276,6 +276,11 @@ internal extension API {
                         return
                     }
                     components.queryItems = params
+                    components.percentEncodedQuery = components.percentEncodedQuery?
+                        .replacingOccurrences(
+                            of: "+",
+                            with: "%2B"
+                        )
 
                     guard let urlComponents = components.url else {
                         let error = ParseError(code: .otherCause,

Sources/ParseSwift/API/API+NonParseBodyCommand.swift

@@ -78,6 +78,11 @@ internal extension API {
                                                message: "Could not unwrap url components for \(url)"))
                 }
                 components.queryItems = params
+                components.percentEncodedQuery = components.percentEncodedQuery?
+                    .replacingOccurrences(
+                        of: "+",
+                        with: "%2B"
+                    )
 
                 guard let urlComponents = components.url else {
                     return .failure(ParseError(code: .otherCause,

Sources/ParseSwift/LiveQuery/ParseLiveQuery.swift

@@ -154,6 +154,11 @@ Not attempting to open ParseLiveQuery socket anymore
             throw error
         }
         components.scheme = (components.scheme == "https" || components.scheme == "wss") ? "wss" : "ws"
+        components.percentEncodedQuery = components.percentEncodedQuery?
+            .replacingOccurrences(
+                of: "+",
+                with: "%2B"
+            )
         url = components.url
         self.task = await URLSession.liveQuery.createTask(self.url,
                                                           taskDelegate: self)

Tests/ParseSwiftTests/APICommandTests.swift

@@ -407,6 +407,55 @@ class APICommandTests: XCTestCase {
         }
     }
 
+    func testQueryWhereEncoding() async throws {
+        let query = Level.query("name" == "[email protected]")
+        let parameters = try query.getQueryParameters()
+
+        let queryCommand = API.NonParseBodyCommand<Query<Level>, Level?>(
+            method: .GET,
+            path: query.endpoint,
+            params: parameters
+        ) { _ in
+            return nil
+        }
+
+        switch await queryCommand.prepareURLRequest(options: []) {
+
+        case .success(let request):
+            XCTAssertEqual(
+                request.url?.absoluteString,
+                "http://localhost:1337/parse/classes/Level?limit=100&skip=0&where=%7B%22name%22:%[email protected]%22%7D"
+            )
+        case .failure(let error):
+            XCTFail(error.localizedDescription)
+        }
+    }
+
+    func testQueryWhereEncodingPlus() async throws {
+        let query = Level.query("name" == "[email protected]")
+        let parameters = try query.getQueryParameters()
+
+        let queryCommand = API.NonParseBodyCommand<Query<Level>, Level?>(
+            method: .GET,
+            path: query.endpoint,
+            params: parameters
+        ) { _ in
+            return nil
+        }
+
+        switch await queryCommand.prepareURLRequest(options: []) {
+
+        case .success(let request):
+            XCTAssertEqual(
+                request.url?.absoluteString,
+                // swiftlint:disable:next line_length
+                "http://localhost:1337/parse/classes/Level?limit=100&skip=0&where=%7B%22name%22:%22test%[email protected]%22%7D"
+            )
+        case .failure(let error):
+            XCTFail(error.localizedDescription)
+        }
+    }
+
     func testClientKeyHeader() async throws {
         guard let clientKey = ParseSwift.configuration.clientKey else {
             throw ParseError(code: .otherCause, message: "Parse configuration should contain key")