Skip to content

Commit 17f5487

Browse files
committed
cleaning up
1 parent 061d71b commit 17f5487

9 files changed

+95
-226
lines changed

.editorconfig

+5-7
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,9 @@
11
root = true
22

33
[*]
4-
charset = utf-8
5-
end_of_line = lf
6-
insert_final_newline = true
4+
charset = utf-8
5+
end_of_line = lf
6+
insert_final_newline = true
77
trim_trailing_whitespace = true
8-
9-
[composer.json]
10-
indent_style = space
11-
indent_size = 4
8+
indent_style = space
9+
indent_size = 4

.scrutinizer.yml

+1-2
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@ filter:
55
- tests/*
66
- examples/*
77
- src/Deprecated/*
8-
- src/Provider/WIP/*
98
- src/Thirdparty/*
109

1110
tools:
@@ -16,7 +15,7 @@ tools:
1615
checks:
1716
php:
1817
code_rating: true
19-
duplication: true
18+
duplication: false
2019
avoid_multiple_statements_on_same_line: true
2120
avoid_fixme_comments: true
2221
avoid_todo_comments: true

CHANGELOG.md

-122
Original file line numberDiff line numberDiff line change
@@ -3,125 +3,3 @@ Hybridauth Change log
33

44
3.1.0 - TBD
55
...
6-
7-
2.3.0 - Nov 16 2014
8-
#87 - Make logging level based
9-
#102 - Uses Exceptions instead of die() to handle critical problems.
10-
#247 - Disable autoloading when doing class_exists()
11-
#111 - Add more informative messages and logging
12-
#221 - Google Provider: Scopes updated
13-
#261 - Check if $params is_array
14-
#265 - OAuth multipart support added
15-
#266 - Twitter Provider: setUserStatus() message and picture support
16-
#267 - fixed adapter undefined indexes and illegal string offsets (issues #200 and #105)
17-
#270 - providers setUserStatus() return status info
18-
#271 - new: provider::getUserStatus($id) retrieves status info (Facebook and Twitter)
19-
#272 - facebook::setUserStatus on a user's Page
20-
#273 - better profile image size for Google and Twitter
21-
#241 - install.php was deleted
22-
#291 - updated links in readmes and removed mentions of install.php.
23-
#294 - Correctly break; when looping on $response->placesLived when a primary location is defined
24-
#295 - Check for callers URL protocol
25-
#289 - Added Facebook language locale (#289)
26-
#293 - Global 'force' configuration option to always request login credentials.
27-
#298 - Add check for OAuthException class exists
28-
#301 - getUserContacts gets paged Facebook friends
29-
#305 - Require wrapper class file only if it is specified
30-
#306 - fixed E_NOTICE: Undefined index: debug_mode in Logger class
31-
#310 - GitHub auth: Undefined property: stdClass::$email
32-
#264 - Yahoo login not returning email address
33-
#321, #322, #323 - Updated QQ
34-
#324 - fixed port issue on openid
35-
36-
2.2.2 - July 11 2014
37-
#239 Fixes to Disqus provider, which now works properly (the old one was not working)
38-
#234 Error fix for when you don't have friends in G+ and try using getUsersContacts()
39-
#226 Fixing fatal error in Yammer Provider
40-
#227 + #229 Github: getUserProfile() does not contain email address fix
41-
#236 + #237 OAuth1Client: Allow to set a request body and a content type
42-
#241 Updates to Install.php
43-
#208 Updating and fixing examples
44-
#244 OAuth if the class already exists, use the current class
45-
#246 Fixing version numbers
46-
Few small fixes and updates to the examples
47-
48-
2.2.1 - May 26 2014
49-
#213 - Hotfix for undefined variable
50-
#215 - Hotfix for BYO Storage Mechanism
51-
52-
2.2.0 - May 24 2014
53-
New provider: Yammer
54-
New provider: freeagent.com
55-
New provider: Drupal OAuth 2
56-
New provider: PayPal OAuth 2
57-
New provider: PayPal OpenID
58-
Removed provider: MySpace
59-
Fixes and updates to the following providers: Foursquare, Yandex, Disqus, Vkontakte, Steam, Odnoklassniki.ru, LinkedIn, Facebook, Twitter
60-
Log curl_exec error for failure tracing
61-
If log file does not exists it will be created
62-
Typo fixes
63-
Many other little changes and fixes
64-
Enabled to have your own storage mechanism #164
65-
66-
2.1.1 - 29 dec 2012
67-
#59 - Remove OAuth PECL conflict with OAuth PHP Library
68-
#70 - Update Facebook PHP SDK (v.3.2.2)
69-
#63 - Minor change to fix getCurrentUrl()
70-
#73 - Fix misspellings of "authentication" and "valid"; me english bad ;
71-
#48 + #62 + #71 - Minor change to remove PHP Notices
72-
#57 - Composer Require CURL
73-
#53 - Proper handling of LinkedIn user profiles with no telephone numbers
74-
#64 - Allow restricting google provider to specific domain
75-
Update Windows Live endpoint URLs (http://msdn.microsoft.com/en-us/library/live/hh243647.aspx)
76-
Update lightopenid library
77-
78-
2.1.0 - 04 aug 2012
79-
Yahoo! provider adapter is now based on OAuth1 protocol (by Lukasz Koprowski)
80-
Windows Live provider adapter is now based on OAuth2 protocol (by Lukasz Koprowski)
81-
Yahoo! and Google openid based are now part of the "additional providers package"
82-
Add proxy config for providers API (by fedetorre)
83-
Fix Hybrid_Providers_Google::getUserContacts()
84-
Fix Twitter user's profile url
85-
Fix the user profile url on Hybrid_Providers_Facebook::getUserActivity()
86-
87-
2.0.11 - 25 jan 2012
88-
Fix a bug with Twitter update status
89-
Add a way to specify adapters and wrappers from config
90-
Added emailVerified property to profile object.
91-
Added ability to pass display type to Facebook SDK.
92-
Refactored endpoint
93-
Add user contacts list for Google Hybrid_Providers_Google::getUserContacts()
94-
Add Hybrid_Auth::getProviders() to return array listing all enabled providers as well as a flag if you are connected.
95-
96-
2.0.10 - 05 dec 2011
97-
Fix a bug with Facebook update status setUserStatus()
98-
Fix a bug with linkedin get contacts list getUserContacts()
99-
Fix refresh token issue with oauth2 client
100-
Remove https://www.googleapis.com/auth/plus.me from the default Google scope
101-
Fix an issue with openid identifier verification
102-
Fix an issue where a user cancel the auth it will persist for some providers
103-
Another numbers of bug fixes and improvements
104-
105-
2.0.9 - 24 nov 2011
106-
Move Google to work with OAuth 2 protocol.
107-
Get back Foursquare and AOL to library core as supported networks
108-
Add 2 new examples "Sign-in/Sign-up Users" and "Basic Facebook integration"
109-
Fix a bug with linkedin update status setUserStatus()
110-
111-
2.0.8 - 18 nov 2011
112-
Add an auto installer to make life a bit easier.
113-
Change the configuration file name from hybridauth.php to config.php
114-
Setting facebook scope from the configuration, now override the default requested scope
115-
Fix a bug with twitter update status setUserStatus()
116-
Fix a bug with logoutAllProviders()
117-
Hybrid_Auth_Activity::$date return now a timestamp across supported social networks
118-
Introduce the refresh_token as its needed for some providers
119-
fixed some minor issues and improved bunch of other stuff
120-
Thanks for KVC, RB Lin and danhstevens!
121-
122-
2.0.7 - 12 oct 2011
123-
Add some generic functions: Hybrid_Provider_Adapter::getUserContacts(), Hybrid_Provider_Adapter::setUserStatus() and Hybrid_Provider_Adapter::getUserActivity() to Facebook, MySpace, Twitter, Identica and LinkedIn.
124-
Add a generic function to logout all connected provider at once Hybrid_Auth::logoutAllProviders()
125-
Correct a bug with Hybrid_Auth::getCurrentUrl()
126-
Split HybridAuth into 2 packages. i) a core library which come with 8 major providers by default, ii) the others one which containt some additional providers
127-
fixed some minors issues and improved some stuff and whatnot

CONTRIBUTING.md

-50
This file was deleted.

COPYING.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ Except where otherwise noted in the source code (i.e., LightOpenID and OAuth
22
Library, which are covered by similar licences but with different Copyright
33
notices) all the files are:
44

5-
Copyright (C) 2009-2014, HybridAuth authors. All Rights Reserved.
5+
Copyright (C) 2009-2015, HybridAuth authors. All Rights Reserved.
66

77
Permission is hereby granted, free of charge, to any person obtaining a copy
88
of this software and associated documentation files (the "Software"), to deal

README.md

+4-6
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
1-
## [Hybridauth](http://hybridauth.sourceforge.net/) 3.0.0-Remake
1+
## [Hybridauth](http://hybridauth.github.io/) 3.0.0-Remake
22

3-
[![Build Status](https://travis-ci.org/hybridauth/hybridauth.svg?branch=3.0.0-Remake)](https://travis-ci.org/hybridauth/hybridauth) [![Scrutinizer Code Quality](https://scrutinizer-ci.com/g/hybridauth/hybridauth/badges/quality-score.png?b=3.0.0-Remake)](https://scrutinizer-ci.com/g/hybridauth/hybridauth/?branch=3.0.0-Remake) [![Latest Stable Version](https://poser.pugx.org/hybridauth/hybridauth/v/stable.png)](https://packagist.org/packages/hybridauth/hybridauth) [![Latest Unstable Version](https://poser.pugx.org/hybridauth/hybridauth/v/unstable.svg)](https://packagist.org/packages/hybridauth/hybridauth) [![Total Downloads](https://poser.pugx.org/hybridauth/hybridauth/downloads.png)](https://packagist.org/packages/hybridauth/hybridauth) [![License](https://poser.pugx.org/hybridauth/hybridauth/license.svg)](https://packagist.org/packages/hybridauth/hybridauth)
3+
[![Build Status](https://travis-ci.org/hybridauth/hybridauth.svg?branch=3.0.0-Remake)](https://travis-ci.org/hybridauth/hybridauth) [![Scrutinizer Code Quality](https://scrutinizer-ci.com/g/hybridauth/hybridauth/badges/quality-score.png?b=3.0.0-Remake)](https://scrutinizer-ci.com/g/hybridauth/hybridauth/?branch=3.0.0-Remake) [![Latest Stable Version](https://poser.pugx.org/hybridauth/hybridauth/v/stable.png)](https://packagist.org/packages/hybridauth/hybridauth) [![Total Downloads](https://poser.pugx.org/hybridauth/hybridauth/downloads.png)](https://packagist.org/packages/hybridauth/hybridauth) [![License](https://poser.pugx.org/hybridauth/hybridauth/license.svg)](https://packagist.org/packages/hybridauth/hybridauth)
44
[![Join the chat at https://gitter.im/hybridauth/hybridauth](https://badges.gitter.im/Join%20Chat.svg)](https://gitter.im/hybridauth/hybridauth?utm_source=badge&utm_medium=badge&utm_campaign=pr-badge&utm_content=badge)
55

66

@@ -144,13 +144,11 @@ $ php composer.phar install
144144

145145
Hybridauth is a community driven project and accepts contributions of code and documentation from the community.
146146

147-
For more information, see [CONTRIBUTING.md](https://github.com/hybridauth/hybridauth/blob/master/CONTRIBUTING.md).
147+
For more information, see http://hybridauth.github.io/getinvolved.html.
148148

149149
#### Questions, Help and Support?
150150

151-
For general questions (i.e, "how-to" questions), consider using [StackOverflow](https://stackoverflow.com/questions/tagged/hybridauth).
152-
153-
We also have a mailing list at [Google Groups](http://groups.google.com/group/hybridauth).
151+
For general questions (i.e, "how-to" questions), please consider using [StackOverflow](https://stackoverflow.com/questions/tagged/hybridauth) instead of the Github issues tracker. For convenience, we also have a [low-activity] mailing list at [Google Groups](http://groups.google.com/group/hybridauth) and a [Gitter channel](https://gitter.im/hybridauth/hybridauth) if you want to get help directly from the community.
154152

155153
#### Project maintainers
156154

TODO.md

+82-30
Original file line numberDiff line numberDiff line change
@@ -1,30 +1,82 @@
1-
#### ToDo
2-
3-
- [ ] Improve this todo list #451 issuecomment-96178499
4-
- [ ] Fix eventual bugs (priority to stability issues)
5-
- [x] Make hybridauth PSR-4 compliant
6-
- [x] Replace hybridauth examples with basic and simple ones.
7-
- [x] Move additional-providers inside core.
8-
- [x] Remove static vars and methods.
9-
- [x] Restructure hybridauth directories and files.
10-
- [x] normalize configuration files (one set of rules for extra arguments).
11-
- [x] Add a HttpClient and eventually support externals libs (i.g., Guzzle)
12-
- [x] Give more consistent and readable errors when requests fails.
13-
- [x] Optimize how hybridauth use php sessions - it store a lot of data
14-
- [x] Do not store empty tokens
15-
- [x] Remove unused configs and params when initiating auth protocols
16-
- [x] No longer serialize exceptions in session.
17-
- [x] Rework OAuth1 and OAuth2 templates
18-
- [x] Merge OAuth1Client with Model_OAuth1
19-
- [x] Merge OAuth2Client with Model_OAuth2
20-
- [x] Rework Exceptions
21-
- [x] Implement Data parser and Data Collection.
22-
- [ ] Upgrading supported providers to 3.0.
23-
- [x] Remove introduced methods and users profile fields.
24-
- [x] Drop support for few additional providers, due to either external dependencies
25-
- [x] Attempt to create non dependent Facebook adapter (Using Model_OAuth2 instead of SDK).
26-
- [ ] Attempt to create non dependent Linkedin adapter (Using Model_OAuth1 instead of simple-linkedinphp. simple-linkedinphp seems to be abandoned).
27-
- [ ] Reduce the supported providers code complexity.
28-
- [ ] Improve internal code comments.
29-
- [ ] Introduce unit testing
30-
- [ ] ..
1+
Hybridauth V3.1 TODO list
2+
=========================
3+
4+
### Required:
5+
6+
[X] **Core**
7+
8+
Hybridauth core has been entirely implemented and while new modification may occur in the future, the design is most likely
9+
will stay the same.
10+
11+
[ ] **OAuth 1/2 Providers**
12+
13+
For the first releases of Hybridauth V3, we'll focusing on the major providers (i.g, facebook, google, twitter, etc.)
14+
15+
The following IDPs are based on either OAuth 1 or 2 specifications. While most of them are already implemented to work with
16+
Hybridauth V3, they still need some tweaking and testing.
17+
18+
More information on how to implement IDPs into Hybridauth V3 can be found at
19+
http://hybridauth.github.io/developer-ref-extend-hybridauth.html
20+
21+
Provider | Implemented? | Working? | Notes
22+
------------- | ------------ |------------ | ------------------------------------------------------------------------------
23+
Facebook | [X] | [ ] |
24+
Twitter | [X] | [ ] |
25+
Google | [X] | [ ] |
26+
Yahoo | [ ] | [ ] | Not implemented yet.
27+
Windows Live | [ ] | [ ] | Not implemented yet.
28+
LinkedIn | [ ] | [ ] | Not implemented yet.
29+
Foursquare | [X] | [ ] |
30+
Disqus | [X] | [ ] |
31+
Dribbble | [X] | [ ] |
32+
GitHub | [X] | [ ] |
33+
Instagram | [X] | [ ] |
34+
Reddit | [X] | [X] |
35+
WordPress | [X] | [X] |
36+
Tumblr | [X] | [ ] |
37+
TwitchTV | [X] | [ ] |
38+
Vkontakte | [X] | [ ] |
39+
Mailru | [X] | [ ] |
40+
Odnoklassniki | [X] | [ ] |
41+
42+
43+
[ ] **OpenID Providers**
44+
45+
The following IDPs are based on OpenID specifications.
46+
47+
Provider | Implemented? | Working? | Notes
48+
------------- | ------------ |------------ | ------------------------------------------------------------------------------
49+
OpenID | [X] | [X] | Generic OpenID Adapter. Implemented and Working.
50+
AOL | [X] | [ ] | Does the AOL still a thing? We may drop it otherwise.
51+
PaypalOpenID | [X] | [ ] | Implemented but needs someone to confirm it's a working as expected.
52+
Stackoverflow | [X] | [X] | All good.
53+
YahooOpenID | [X] | [X] | All good.
54+
Steam | [X] | [X] | Steam Adapter is a mix of OpenID and a Proprietary API.
55+
56+
57+
### Optional:
58+
59+
The following tasks are not required in order to release Hybridauth 3 but they're nice to have.
60+
61+
[ ] **Peer review**
62+
63+
Hybridauth V3 is a complete rewrite of the current V2, and given the sensitive topic it deals with (i.e, users authentication),
64+
it would be extremely valuable to have the entire code base examined and reviewed for safety and security purposes.
65+
66+
[ ] **Additional providers**
67+
68+
As mentioned above, for the first releases of Hybridauth 3, we'll focusing on the major providers. If there's a social networks
69+
that you care for and you wish it to be included sooner, then feel free to port it from the V2 repository.
70+
71+
V2 additional IDPs can be found at:
72+
73+
https://github.com/hybridauth/hybridauth/tree/master/additional-providers
74+
75+
Documentation on how to upgrade these providers to V3 can be found at:
76+
77+
http://hybridauth.github.io/developer-ref-extend-hybridauth.html
78+
79+
[ ] **Unit tests**
80+
81+
This one has been on the project's wishlist for as long as Hybridauth library existed. If you feel adventurous enough to take
82+
on this task, then please give it a shot.

UPGRADING.md

-4
This file was deleted.

composer.json

+2-4
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
"name": "hybridauth/hybridauth",
33
"description": "Open source social sign on PHP library.",
44
"keywords": ["oauth", "openid", "authentication", "authorization", "social", "api", "google", "facebook", "twitter"],
5-
"homepage": "http://hybridauth.sourceforge.net",
5+
"homepage": "http://hybridauth.github.io",
66
"type": "library",
77
"license": "MIT",
88
"authors": [
@@ -12,7 +12,6 @@
1212
}
1313
],
1414
"support": {
15-
"forum": "https://groups.google.com/d/forum/hybridauth",
1615
"issues": "https://github.com/hybridauth/hybridauth/issues",
1716
"gitter": "https://gitter.im/hybridauth/hybridauth"
1817
},
@@ -21,8 +20,7 @@
2120
"ext-curl": "*"
2221
},
2322
"require-dev": {
24-
"phpunit/phpunit": "~4.1",
25-
"fabpot/php-cs-fixer": "~1.6"
23+
"phpunit/phpunit": "~4.1"
2624
},
2725
"autoload": {
2826
"psr-4": {

0 commit comments

Comments
 (0)