Skip to content

Commit 22cc45e

Browse files
committed
Log entire NSError instead of just localizedDescription
Some NSErrors being logged don't have a NSLocalizedDescriptionKey in the userInfo so that logging its localizedDescription gives no details. We want to give developers adequate information. Since these are just logs and we want to provide more information, instead let's log the entire error object.
1 parent 0348dd1 commit 22cc45e

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)