Hello João,
After upgrading our InAppBrowser plugin, I noticed that the IdPMobile stopped working and then I upgraded it also to the most recent version.
But even after upgrading, it was not working. After debugging it a few times, I think I found a bug in the OnBrowserPageNavigationCompleted handler (action InAppBrowserOnLoadStart).
In the evt_url parameter, the module is using the URL variable (all uppercase), but this is a local variable that never changes, something like "IdP/Login.aspx?FromMobile=True&AppConfigId=1&ForceAuth=False".
The correct variable here should be Url (pascal Case), which is in fact the variable sent from the event triggered by the InAppBrowserEvents. Using the URL variable my InAppBrowser was never closing and I was stuck in the "IdP/MobileCloseInAppPoint.aspx" page.
I'd be glad to hear from you about this.
And by the way, thanks for providing this great solution =)
Hi @Rande Arievilo Moreira ,
Thank you for reporting this issue, however, I was double checking and after downloading the version published to the Forge I was unable to find references to the usage of the "URL" upper case on the callback actions of the events.I suspect that might have been due to the component upgrade order, I'm not sure.
In any case I believe it will be better for me to push a new version with distinct variable names to prevent this from happening again.
Hello @João Barata ,
What you said might make sense. I checked the original downloaded version and also my module revision history and it seems that at some point the "Url" variable was replaced by the "URL", maybe due to some "automagic" mapping that Service Studio did during the upgrade process.
I believe that renaming the variable to avoid confusion will solve it for future versions.
Thanks!