-
Notifications
You must be signed in to change notification settings - Fork 2.3k
Failed to create Short Dynamic Link on iOS #1702
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Comments
me too! I think maybe this issues can help you |
@mf-fengsheng Thanks for the tip, but unfortunately it didn't solve my issue. I did however find the reason why it fails. The URL shortener service of Firebase require a API key to be set. This key is defined in I'm not sure if the problem lies in |
The issue is not with |
@forsen how can I fix it? Please show the solution. |
@dantn93 the problem I had was fixed by firebase/firebase-ios-sdk#2124, and a suitable workaround was described in #1702 (comment). Make sure your code finds your API key (as it is required by the url shortener service). If you're on a enterprise network: |
@forsen i am using react native firebase 5.5.5, but still i am getting this error in ios, android its working |
I decided to use google api, the rn firebase deeplink is bad
…On Sun, Jul 21, 2019 at 2:00 PM affanhashone ***@***.***> wrote:
@forsen <https://github.com/forsen> i am using react native firebase
5.5.5, but still i am getting this error in ios, android its working
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#1702?email_source=notifications&email_token=AE3VM2Z2G7QA2NNGHERHO4LQAQCRNA5CNFSM4GF36DHKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2N5KAI#issuecomment-513529089>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AE3VM24DRMKAMZARQFNLY33QAQCRNANCNFSM4GF36DHA>
.
|
@dantn93 Thank You Somuch for hinting out that. |
I've just spent a good half an hour to figure out why it cannot create the short link, until I figured out the page link needs to contain
|
This is indicated here https://rnfirebase.io/docs/v5.x.x/releases/v5.5.x#5.5.0 and here https://rnfirebase.io/docs/v5.x.x/releases/v5.5.x#Upgrade-instructions and https://rnfirebase.io/docs/v5.x.x/links/reference/DynamicLink#Constructor - I'm not sure what more we can do? That said, @markhomoki can you confirm that this works if you use https? |
@markhomoki I was having same thing in code, but still i couldn't create shortLink In IOS. SO i move to REST Full Method |
Yes, in my case it's working fine using |
That code didn't change after v5.5.0 that I'm aware of. It may be that there are further configurations required if you use more custom links vs page.link - I haven't implemented this in my app myself but I recall there were all sorts of things that needed to be done in google and apple-specific ways to enable really custom links. Really vague comment but main point is that maybe page.link works and others don't @affanhashone were you trying xyz.page.link or something different? |
@mikehardy abc123.page.link this was the example, but i created my own page link, i used with https, and it was working only in android, not on ios |
Hmm @affanhashone I would suspect some sort of config issue then, if I recall the config on the iOS side went as far as entitlements or some such, but as I mentioned I haven't integrated it myself so this is necessarily vague, sorry |
@mikehardy maybe some configuration issue, but couldnt figure out. so now i am using firebase rest api. |
In our case the issue was that there was a We can probably put a PR up that gets rid of extra slashes at the end... It took me quite a while and Xcode debugging to find the cause. |
On v6 we should probably handle these better. Does anyone know the exact rules for |
I'm using https:// and still fail with createShortDynamicLink |
Thanks alot bro. made my day |
Docs are open sources with edit button on each page, please make PR to docs |
Issue
When trying to create a short dynamic link on iOS I get an exception from
ExceptionsManager
. Creating a regular dynamic link works as expected. Android works as expected.Which produce the following output in my console:
For some reason,
createShortDynamicLink
produces the exception in the first part of the log above, whilecreateDynamicLink
produces the expected output in the last line of the log above.note: anything identifying our project is anonymised (replaced actual domains and identifiers with
example.com
). We are using the correctpage.link
, not only does it work to create a long dynamic link, but the rest of deep linking functionality works as expected.Project Files
iOS
ios/Podfile
:AppDelegate.m
:Android
android/build.gradle
:android/app/build.gradle
:android/settings.gradle
:MainApplication.kt
:AndroidManifest.xml
:Environment
ios 12
10.14.1
N/A
N/A
Xcode 10.1
React Native
version:0.57.4
React Native Firebase
library version:5.1.0
Firebase
module(s) you're using that has the issue:TypeScript
?N/A
ExpoKit
?ExpoKit
N/A
Think
react-native-firebase
is great? Please consider supporting the project with any of the below:React Native Firebase
andInvertase
on TwitterThe text was updated successfully, but these errors were encountered: