Closed briannovillo closed 7 years ago
Hi guys, I took the dare to update the Linkedin endpoints for show the new popup because has a best UI and because the security workflow is more complete, for example, when we use the login from an emulator, this send an email to the user with a code advising that is an unknown device.
I hope this change to be useful for the community.
Regards, Brian Novillo.
Just to confirm, you've tested this change and it works fine? Happy to merge it once you confirm.
Best,
Sure! My Ionic app is already using this fix and it works fine.
Thanks for contributing!
Added headers: 'Access-Control-Allow-Origin': '' 'X-Requested-With': 'XMLHttpRequest'
Headers were added for avoid ionic CORS error "XMLHttpRequest cannot load linkedin.com/uas/oauth2/accessToken", mentioned in different sites, see:
https://github.com/nraboy/ng-cordova-oauth/issues/32 http://stackoverflow.com/questions/21630081/linkedin-button-xmlhttprequest-cannot-load http://stackoverflow.com/questions/23823510/xmlhttprequest-cannot-load-https-www-linkedin-com-origin-http-xx-xx-i http://stackoverflow.com/questions/23580772/client-server-no-access-control-allow-origin-when-returning-json-from-own-doma