codenitive / laravel-oneauth

OAuth and OAuth2 Auth bundle for Laravel
http://bundles.laravel.com/bundle/oneauth
MIT License
86 stars 16 forks source link

Fix for Windows Live ID Service Provider #36

Closed stayallive closed 11 years ago

stayallive commented 11 years ago

I needed to use Windows Live ID on a project and I soon found out it is pretty broken!

So I hereby provide a fix. If I need to make changes to this please let me know!

Change logs:

crynobone commented 11 years ago

I feel it best to use text instead of string for 'acces_token', can you make the update?

stayallive commented 11 years ago

Will do first thing in the morning!

It was my initial thought but the access_token column is indexed... is that even needed in the first place?

crynobone commented 11 years ago

Don't think so, most request would look for uid and provider. I must have added that for unneccesary purpose

stayallive commented 11 years ago

Changed it :)

crynobone commented 11 years ago

Thanks