Skip to content

Commit 7d94ac2

Browse files
committed
Fix Private API
• In a recent update, we added a method called valueForQueryParameter() that extracted URL query parameters from URLs • It appears that Apple has a private API with a similar name and blocks all app uploads using this method name... • Resolved by renaming to valueFromQueryParameter()
1 parent 8b36ad0 commit 7d94ac2

File tree

3 files changed

+3
-3
lines changed

3 files changed

+3
-3
lines changed

iOS_SDK/OneSignalSDK/Source/NSURL+OneSignal.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,6 @@
2929

3030
@interface NSURL (OneSignal)
3131

32-
- (NSString *)valueForQueryParameter:(NSString *)parameter;
32+
- (NSString *)valueFromQueryParameter:(NSString *)parameter;
3333

3434
@end

iOS_SDK/OneSignalSDK/Source/NSURL+OneSignal.m

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@
2828
#import "NSURL+OneSignal.h"
2929

3030
@implementation NSURL (OneSignal)
31-
- (NSString *)valueForQueryParameter:(NSString *)parameter {
31+
- (NSString *)valueFromQueryParameter:(NSString *)parameter {
3232
NSURLComponents *components = [NSURLComponents componentsWithURL:self resolvingAgainstBaseURL:false];
3333

3434
for(NSURLQueryItem *item in components.queryItems)

iOS_SDK/OneSignalSDK/Source/OneSignalHelper.m

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -767,7 +767,7 @@ + (NSString*)downloadMediaAndSaveInBundle:(NSString*)urlString {
767767
if (mimeType != nil && ![mimeType isEqualToString:@""]) {
768768
newExtension = mimeType.fileExtensionForMimeType;
769769
} else {
770-
newExtension = [[[NSURL URLWithString:urlString] valueForQueryParameter:@"filename"] supportedFileExtension];
770+
newExtension = [[[NSURL URLWithString:urlString] valueFromQueryParameter:@"filename"] supportedFileExtension];
771771
}
772772

773773
if (!newExtension || ![ONESIGNAL_SUPPORTED_ATTACHMENT_TYPES containsObject:newExtension])

0 commit comments

Comments
 (0)