Skip to content
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

Branch modifying data inside deep link when +match_guaranteed is FALSE #1220

Open
arthurgiani opened this issue Sep 11, 2024 · 1 comment
Open

Comments

@arthurgiani
Copy link

Describe the bug

This issue was originally posted on the RodrigoSMarques/flutter_branch_sdk#379.

When listening to the deep link, if deepLink[+match_guaranteed] is false, branch deep link is modifying data inside.

Steps to reproduce

The following flow is being made:

1)A deep link is created through the Branch API.
2)Through the API, I added a key-pair 'params' attribute containing some important info for my business rule on this link.
3)This link is sent to a random user (that does not have the app installed).
4)The user clicks on the link and download the app.
5)The app listens to the deep link.

Now, imagine that this 'params' attribute that I described on 2) contains an integer as one of the values. In this scenario, when +match_guaranteed=false, this attribute is being sent through FlutterBranchSdk.listSession() as a DOUBLE, not an int.

Screenshots
Screenshot 1: event response when +match_guaranteed is FALSE (when app opens for the first time after the install)
On the deep link creation, eventId is 0 (int), and not 0.0 (double)
366225160-5164a049-75ab-4039-905c-086028bf19ad

Screenshot 2: event response when +match_guaranteed is TRUE (when I click on the deep link with the app installed)
On the deep link creation, eventId is 0 (int), exactly the way I settled on the backend.
366225765-b959aa4c-c648-4b4f-9d9b-c38be49449aa

Expected behavior

Expected behavior
Branch link should always show exactly the same information as when they are created.

SDK Version

5.12.2

Make and Model

Samsung S21

OS

14

Additional Information/Context

No response

Copy link

This issue has been automatically marked as stale due to inactivity for 60 days. If this issue is still relevant, please respond with any updates or this issue will be closed in 7 days. If you believe this is a mistake, please comment to let us know. Thank you for your contributions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant