-
Notifications
You must be signed in to change notification settings - Fork 296
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
Improve Adjust.processAndResolve method error handling #744
Comments
Same problem. |
Hey guys, Thank you for the report. Is this something you are able to reproduce on your end? Any example of the shortened URL which you send and never get back the resolved URL would be of great help to be able to check this on our end. Also, @tdesert, thank you for your suggestion, it makes sense and we'll look into adding these changes to cover these cases as well. |
@uerceg Could you give me an email account? We are using our company account and would like to give you an example by mail. |
Sure thing, you can find it if you check my GitHub profile. |
@uerceg Thanks! I sent the email now. |
Hello,
We've noticed using async version of
Adjust.processAndResolve()
that several situations lead to this method never completing:Adjust.disable()
has been called (e.g. when user rejected Adjust from CMP)Here is a pseudo code example of where is can occur:
As we understand this method call should be avoided in the identified situations, we believe it would be safer that the SDK guarantees this method always completes, being successful or not.
The reason being: as soon as we decide to work with Adjust link resolution, all the deeplinks and universal links will have to go through this resolution method before being able to handle the user redirection.
As other situations could lead to unsuccessful redirection than the 2 we have identified above, we need our app to be able to handle this situation gracefully and avoid the worst case scenario where the user is never redirected.
Would it be possible to improve the
processAndResolve
method with one of those 2 options?Adjust.processAndResolve
returnsnil
when the resolution failed for any reasonAdjust.processAndResolve
throws an error when resolution failed for any reasonSide note: we could not find reference to
Adjust.processAndResolve
method in documentation, which refers toresolveLinkWithUrl
method that seems to no longer exist in latest Adjust SDK version.Thanks for you support.
The text was updated successfully, but these errors were encountered: