facebook-ios-sdk icon indicating copy to clipboard operation
facebook-ios-sdk copied to clipboard

Improper Swizzling of `application:openURL:options:`

Open waltflanagan opened this issue 2 years ago • 12 comments

Checklist before submitting a bug report

Xcode version

14.3

Facebook iOS SDK version

16.1.0

Dependency Manager

SPM

SDK Framework

Core

Goals

After updating to SDK 16.* we found that application:openURL:options: was always returning NO/false when called. This is due to swizzling implemented here. The return value is not passed through this implementation.

We would like to have callers of application:openURL:options: receive the correct return value of the method.

Expected results

If the implementation of application:openURL:options: returns YES then the callers of the method should receive YES

Actual results

Callers of application:openURL:options: always receive NO.

Steps to reproduce

Implement a UIApplicationDelegate class and hard core application:openURL:options: to return YES/true.

Link against FBAEMKit and enable it such that it swizzles application:openURL:options:

Code samples & details

No response

waltflanagan avatar May 08 '23 17:05 waltflanagan

We're experiencing this issue too - it has broken our deep link implementation, so we're going to have to downgrade.

squarefrog avatar Sep 27 '23 09:09 squarefrog

We're experiencing the same issue

shomin1983 avatar Nov 15 '23 07:11 shomin1983

Same Here!

dodatw avatar Nov 15 '23 07:11 dodatw

It is the same story with application(_:continue:restorationHandler:) -> Bool and Facebook SDK 16.3.1

olejnjak avatar Jan 12 '24 00:01 olejnjak

Here is minimal example for my case FBSample.zip

olejnjak avatar Jan 14 '24 00:01 olejnjak

This issue still exists in 17.0.2 Attached is the AppDelegate illustrating the issue AppDelegate.swift.zip

Cedrick84 avatar Aug 04 '24 09:08 Cedrick84