Skip to content

Commit bf0f979

Browse files
authored
Merge pull request #1068 from OneSignal/fix/more_detailed_error_logs
Log entire `NSError` instead of just `localizedDescription`
2 parents 0348dd1 + 22cc45e commit bf0f979

File tree

5 files changed

+5
-5
lines changed

5 files changed

+5
-5
lines changed

iOS_SDK/OneSignalSDK/Source/OSInAppMessageView.m

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,7 @@ - (NSString *)getTagsString {
6767
NSString *jsonString;
6868
if (!jsonData) {
6969
[OneSignal onesignalLog:ONE_S_LL_ERROR message:
70-
[NSString stringWithFormat:@"Error parsing tag dictionary to json :%@",error.localizedDescription]];
70+
[NSString stringWithFormat:@"Error parsing tag dictionary to json: %@", error]];
7171
} else {
7272
jsonString = [[NSString alloc] initWithData:jsonData encoding:NSUTF8StringEncoding];
7373
}

iOS_SDK/OneSignalSDK/Source/OneSignal.m

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1733,7 +1733,7 @@ + (void)trackColdRestart {
17331733
[[OneSignalClient sharedClient] executeRequest:[OSRequestTrackV1 trackUsageData:osUsageData appId:appId] onSuccess:^(NSDictionary *result) {
17341734
[OneSignal onesignal_Log:ONE_S_LL_VERBOSE message:@"trackColdRestart: successfully tracked cold restart"];
17351735
} onFailure:^(NSError *error) {
1736-
[OneSignal onesignal_Log:ONE_S_LL_ERROR message:[NSString stringWithFormat:@"trackColdRestart: Failed to track cold restart: %@", error.localizedDescription]];
1736+
[OneSignal onesignal_Log:ONE_S_LL_ERROR message:[NSString stringWithFormat:@"trackColdRestart: Failed to track cold restart: %@", error]];
17371737
}];
17381738
}
17391739
}

iOS_SDK/OneSignalSDK/Source/OneSignalHelper.m

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -104,7 +104,7 @@ -(void)URLSession:(NSURLSession *)session dataTask:(NSURLSessionDataTask *)dataT
104104
}
105105

106106
if (fileHandleError != nil) {
107-
[OneSignal onesignalLog:ONE_S_LL_ERROR message:[NSString stringWithFormat:@"OneSignal Error encountered while downloading attachment: %@", fileHandleError.localizedDescription]];
107+
[OneSignal onesignalLog:ONE_S_LL_ERROR message:[NSString stringWithFormat:@"OneSignal Error encountered while downloading attachment: %@", fileHandleError]];
108108
}
109109
}
110110

iOS_SDK/OneSignalSDK/Source/OneSignalNotificationServiceExtensionHandler.m

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -157,7 +157,7 @@ + (void)onNotificationReceived:(NSString *)receivedNotificationId withBlockingTa
157157
dispatch_semaphore_signal(semaphore);
158158
}
159159
} failureBlock:^(NSError *error) {
160-
[OneSignal onesignal_Log:ONE_S_LL_VERBOSE message:[NSString stringWithFormat:@"OneSignal onNotificationReceived sendReceiveReceipt Failed for playerId: %@ error:%@", playerId, error.localizedDescription]];
160+
[OneSignal onesignal_Log:ONE_S_LL_ERROR message:[NSString stringWithFormat:@"OneSignal onNotificationReceived sendReceiveReceipt Failed for playerId: %@ error: %@", playerId, error]];
161161
if (semaphore) {
162162
dispatch_semaphore_signal(semaphore);
163163
}

iOS_SDK/OneSignalSDK/Source/OneSignalWebView.m

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -84,7 +84,7 @@ -(void)webView:(WKWebView *)webView didFinishNavigation:(WKNavigation *)navigati
8484
}
8585

8686
-(void)webView:(WKWebView *)webView didFailNavigation:(WKNavigation *)navigation withError:(NSError *)error {
87-
[OneSignal onesignal_Log:ONE_S_LL_ERROR message:error.localizedDescription];
87+
[OneSignal onesignal_Log:ONE_S_LL_ERROR message:[NSString stringWithFormat:@"webView: An error occurred during navigation: %@", error]];
8888
}
8989

9090
- (void)pinSubviewToMarginsWithSubview:(UIView *)subview withSuperview:(UIView *)superview {

0 commit comments

Comments
 (0)