Convert url request to Swift

Author: noahsmartin

CHANGELOG.md

@@ -18,6 +18,7 @@
 - Crashes for uncaught NSExceptions will now report the stracktrace recorded within the exception (#5306)
 - Move SentryExperimentalOptions to a property defined in Swift (#5329)
 - Add redaction in session replay for `SFSafariView` used by `SFSafariViewController` and `ASWebAuthenticationSession` (#5408)
+- Convert SentryNSURLRequest to Swift (#5451)
 
 ### Fixes
 

Sentry.xcodeproj/project.pbxproj

@@ -15,8 +15,6 @@
 #import "SentryEvent.h"
 #import "SentryFileManager.h"
 #import "SentryLog.h"
-#import "SentryNSURLRequest.h"
-#import "SentryNSURLRequestBuilder.h"
 #import "SentryOptions.h"
 #import "SentrySerialization.h"
 #import "SentrySwift.h"
@@ -339,14 +337,13 @@ - (void)sendAllCachedEnvelopes
     rateLimitedEnvelope.header.sentAt = SentryDependencyContainer.sharedInstance.dateProvider.date;
 
     NSError *requestError = nil;
-    NSURLRequest *request = [self.requestBuilder createEnvelopeRequest:rateLimitedEnvelope
-                                                                   dsn:self.options.parsedDsn
-                                                      didFailWithError:&requestError];
+    NSURLRequest *request =
+        [self.requestBuilder createEnvelopeRequestWithEnvelope:rateLimitedEnvelope
+                                                           dsn:self.options.parsedDsn
+                                                         error:&requestError];
 
-    if (nil == request || nil != requestError) {
-        if (nil != requestError) {
-            SENTRY_LOG_DEBUG(@"Failed to build request: %@.", requestError);
-        }
+    if (nil != requestError) {
+        SENTRY_LOG_DEBUG(@"Failed to build request: %@.", requestError);
         [self recordLostEventFor:rateLimitedEnvelope.items];
         [self deleteEnvelopeAndSendNext:envelopeFilePath];
         return;

Sources/Sentry/SentryHttpTransport.m

@@ -4,10 +4,9 @@
 #import "SentryEnvelopeItemHeader.h"
 #import "SentryEnvelopeItemType.h"
 #import "SentryLog.h"
-#import "SentryNSURLRequest.h"
-#import "SentryNSURLRequestBuilder.h"
 #import "SentryOptions.h"
 #import "SentrySerialization.h"
+#import "SentrySwift.h"
 #import "SentryTransport.h"
 
 NS_ASSUME_NONNULL_BEGIN
@@ -66,14 +65,12 @@ - (void)sendEnvelope:(SentryEnvelope *)envelope
                                                                       items:allowedEnvelopeItems];
 
     NSError *requestError = nil;
-    NSURLRequest *request = [self.requestBuilder createEnvelopeRequest:envelopeToSend
-                                                                   url:self.apiURL
-                                                      didFailWithError:&requestError];
+    NSURLRequest *request = [self.requestBuilder createEnvelopeRequestWithEnvelope:envelopeToSend
+                                                                               url:self.apiURL
+                                                                             error:&requestError];
 
-    if (nil == request || nil != requestError) {
-        if (nil != requestError) {
-            SENTRY_LOG_ERROR(@"Unable to build envelope request with error %@", requestError);
-        }
+    if (nil != requestError) {
+        SENTRY_LOG_ERROR(@"Unable to build envelope request with error %@", requestError);
         return;
     }
 

Sources/Sentry/SentryNSURLRequest.m

@@ -4,14 +4,14 @@
 #import "SentryEnvelopeRateLimit.h"
 #import "SentryHttpDateParser.h"
 #import "SentryHttpTransport.h"
-#import "SentryNSURLRequestBuilder.h"
 #import "SentryOptions.h"
 #import "SentryQueueableRequestManager.h"
 #import "SentryRateLimitParser.h"
 #import "SentryRateLimits.h"
 
 #import "SentryRetryAfterHeaderParser.h"
 #import "SentrySpotlightTransport.h"
+#import "SentrySwift.h"
 #import "SentryTransport.h"
 
 NS_ASSUME_NONNULL_BEGIN

Sources/Sentry/SentryNSURLRequestBuilder.m

@@ -0,0 +1,27 @@
+@_implementationOnly import _SentryPrivate
+
+private enum Error: Swift.Error {
+    case serializationError
+}
+
+@objc(SentryNSURLRequestBuilder)
+@_spi(Private) public class SentryURLRequestBuilder: NSObject {
+    
+    @objc
+    public func createEnvelopeRequest(envelope: SentryEnvelope, dsn: SentryDsn) throws -> URLRequest {
+        guard let data = SentrySerialization.data(with: envelope) else {
+            SentryLog.error("Envelope cannot be converted to data")
+            throw Error.serializationError
+        }
+        return try SentryURLRequestFactory.envelopeRequest(with: dsn, data: data)
+    }
+    
+    @objc
+    public func createEnvelopeRequest(envelope: SentryEnvelope, url: URL) throws -> URLRequest {
+        guard let data = SentrySerialization.data(with: envelope) else {
+            SentryLog.error("Envelope cannot be converted to data")
+            throw Error.serializationError
+        }
+        return try SentryURLRequestFactory.envelopeRequest(with: url, data: data, authHeader: nil)
+    }
+}