aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | [feature/passwords] Rename manager methods to check() and hash()Marc Alexander2013-10-022-13/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These method names are more straightforward than the previous ones. PHPBB3-11610
| * | | | | [feature/passwords] Inject passwords manager into auth providersMarc Alexander2013-10-026-20/+82
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The passwords manager will replace the old method of using the functions phpbb_hash() and phpbb_check_hash(). PHPBB3-11610
| * | | | | [feature/passwords] Do not hash passwords longer than 4096 bytesMarc Alexander2013-10-011-0/+14
| | | | | | | | | | | | | | | | | | | | | | | | PHPBB3-11610
| * | | | | [feature/passwords] Pass config via service container to driver helperMarc Alexander2013-09-283-7/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This will get rid of the global $config in the driver helper PHPBB3-11610
| * | | | | [feature/passwords] A few more corrections in methods due to namespacingMarc Alexander2013-09-282-7/+7
| | | | | | | | | | | | | | | | | | | | | | | | PHPBB3-11610
| * | | | | [feature/passwords] More changes due to namespacingMarc Alexander2013-09-272-10/+12
| | | | | | | | | | | | | | | | | | | | | | | | PHPBB3-11610
| * | | | | [feature/passwords] Modify passwords files for namespacing changesMarc Alexander2013-09-2710-26/+44
| | | | | | | | | | | | | | | | | | | | | | | | PHPBB3-11610
| * | | | | [feature/passwords] Add schema changes to schema_data for new hash typesMarc Alexander2013-09-271-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | PHPBB3-11610
| * | | | | Merge branch 'develop' of https://github.com/phpbb/phpbb3 into feature/passwordsMarc Alexander2013-09-27590-7499/+10910
| |\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: phpBB/develop/create_schema_files.php
| * | | | | | [feature/passwords] Use dependency injection for passwords driver helperMarc Alexander2013-09-224-17/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The passwords driver helper is now injected into the driver base instead of being manually loaded. PHPBB3-11610
| * | | | | | [feature/passwords] Use dependency injection for helperMarc Alexander2013-09-224-9/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This will now be used instead of manually loading the passwords helper instance in the passwords manager. PHPBB3-11610
| * | | | | | [feature/passwords] Rename migrations file to passwordsMarc Alexander2013-09-202-63/+38
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Also removed the effectively_installed() check that is not needed according to EXreaction. PHPBB3-11610
| * | | | | | [feature/passwords] Do not manually include passwords drivers in testsMarc Alexander2013-09-201-5/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | PHPBB3-11610
| * | | | | | [feature/passwords] Do not pass phpbb_container to passwords managerMarc Alexander2013-09-209-87/+86
| | | | | | | | | | | | | | | | | | | | | | | | | | | | PHPBB3-11610
| * | | | | | [feature/passwords] Add missing function documentationMarc Alexander2013-09-141-0/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Additionally, a new line has been added to make the code look nicer. PHPBB3-11610
| * | | | | | [feature/passwords] Rename "crypto" files to "passwords" filesMarc Alexander2013-09-1413-135/+135
| | | | | | | | | | | | | | | | | | | | | | | | | | | | PHPBB3-11610
| * | | | | | [feature/passwords] Properly set convert_flag and add test for itMarc Alexander2013-09-142-1/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | PHPBB3-11610
| * | | | | | [feature/passwords] Define default hashing algorithm in container fileMarc Alexander2013-09-144-10/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The default hashing algorithm type is defined as a parameter in the service definition file for the password hashing system. This will allow us to change this in the future but it will also prevent unexperienced admins from changing the hashing algorithm. PHPBB3-11610
| * | | | | | [feature/passwords] Add get_name() method to driversMarc Alexander2013-09-143-0/+36
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This will allow us to actually properly differentiate between the available drivers. PHPBB3-11610
| * | | | | | [feature/passwords] Move migrations file to phpbb folderMarc Alexander2013-09-141-0/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | PHPBB3-11610
| * | | | | | [feature/passwords] Move files after namespacing changesMarc Alexander2013-09-1410-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | PHPBB3-11610
| * | | | | | [feature/passwords] Add legacy support for md5 passwordsMarc Alexander2013-09-143-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | PHPBB3-11610
| * | | | | | [feature/passwords] Increase code coverage in bcrypt driverMarc Alexander2013-09-141-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | PHPBB3-11610
| * | | | | | [feature/passwords] Increase test coverage of salted_md5 driverMarc Alexander2013-09-141-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | PHPBB3-11610
| * | | | | | [feature/passwords] Fix incorrect call to helper instead of $thisMarc Alexander2013-09-141-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The driver helper method unique_id is called inside the helper. Therefore, it shouldn't be called via $this->helper but rather via $this. PHPBB3-11610
| * | | | | | [feature/passwords] Increase manager test coverage to cover all methodsMarc Alexander2013-09-141-2/+42
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | All methods in the manager are now fully covered. PHPBB3-11610
| * | | | | | [feature/passwords] Do not use specific errors but just return falseMarc Alexander2013-09-141-12/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The authentication system should handle the possible errors for now. Additional error returns can be added later on if they are needed. PHPBB3-11610
| * | | | | | [feature/passwords] NULL should be null according to coding guidelinesMarc Alexander2013-09-141-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | PHPBB3-11610
| * | | | | | [feature/passwords] Do not check if type_map is emptyMarc Alexander2013-09-141-11/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If the type map is empty the service container is incomplete or broken. There is no need to check this as other components would probably be broken at the same time. PHPBB3-11610
| * | | | | | [feature/passwords] Use randomly generated password for testsMarc Alexander2013-09-141-28/+36
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The password will start with 'foobar' and then continue with random characters that will be appended every iteration. PHPBB3-11610
| * | | | | | [feature/passwords] Fix tests on PHP versions < 5.3.7Marc Alexander2013-09-141-18/+38
| | | | | | | | | | | | | | | | | | | | | | | | | | | | PHPBB3-11610
| * | | | | | [feature/passwords] Properly treat duplicates in combined hashesMarc Alexander2013-09-142-1/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | PHPBB3-11610
| * | | | | | [feature/passwords] Make method get_hashing_public_algorithm publicMarc Alexander2013-09-141-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Combined hashing requires it for checking the supplied password hash for its hashing algorithm. PHPBB3-11610
| * | | | | | [feature/passwords] Add tests for combined hashing of passwordsMarc Alexander2013-09-141-2/+39
| | | | | | | | | | | | | | | | | | | | | | | | | | | | PHPBB3-11610
| * | | | | | [feature/passwords] Add combined hashing method to helperMarc Alexander2013-09-142-0/+40
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This will allow to hash a previously hashed password with another hashing method, i.e. as upgrade path from phpBB 3.0 to 3.1. PHPBB3-11610
| * | | | | | [feature/passwords] Add function for obtaining only the hash to helperMarc Alexander2013-09-141-0/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is also needed for combined hashing of passwords. PHPBB3-11610
| * | | | | | [feature/passwords] Fix minor code limitations in helperMarc Alexander2013-09-141-2/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These limitations caused it to only allow a specific input to combined hashes. PHPBB3-11610
| * | | | | | [feature/passwords] Pass phpbb_container to passwords helperMarc Alexander2013-09-142-2/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | PHPBB3-11610
| * | | | | | [feature/passwords] Use correct prefix for salt in salted_md5Marc Alexander2013-09-141-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | PHPBB3-11610
| * | | | | | [feature/passwords] Add method for obtaining the hash settings onlyMarc Alexander2013-09-143-0/+38
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is needed for combined hashing of passwords. PHPBB3-11610
| * | | | | | [feature/passwords] Add schema changes for new hash typesMarc Alexander2013-09-149-9/+72
| | | | | | | | | | | | | | | | | | | | | | | | | | | | PHPBB3-11610
| * | | | | | [feature/passwords] Make sure hash has the required lengthMarc Alexander2013-09-142-3/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Also added tests to make sure that crypto drivers are enforcing the hash length. PHPBB3-11610
| * | | | | | [feature/passwords] Make method get_hashing_algorithm() protectedMarc Alexander2013-09-141-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There is currently no use for it being public. PHPBB3-11610
| * | | | | | [feature/passwords] Add tests for manager check_hash() methodMarc Alexander2013-09-141-0/+40
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Tests cover all supported hashing algorithms. PHPBB3-11610
| * | | | | | [feature/passwords] Fix tests for PHP version < 5.3.7Marc Alexander2013-09-141-6/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | PHPBB3-11610
| * | | | | | [feature/passwords] Use common salt generation function for all typesMarc Alexander2013-09-143-31/+47
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We will be using the same method inside the driver helper class for all hash types. This is the same function that has been used for the salted md5 hash of phpBB 3.0. PHPBB3-11610
| * | | | | | [feature/passwords] Add driver helper class fileMarc Alexander2013-09-141-0/+108
| | | | | | | | | | | | | | | | | | | | | | | | | | | | PHPBB3-11610
| * | | | | | [feature/passwords] Include driver helper in manager_testMarc Alexander2013-09-141-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | PHPBB3-11610
| * | | | | | [feature/passwords] Add driver helper class for additional functionsMarc Alexander2013-09-143-28/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Functions for the helper class might be used in other drivers as well and therefore shouldn't be limited to just one driver. PHPBB3-11610
| * | | | | | [feature/passwords] Add basic passwords manager test fileMarc Alexander2013-09-141-0/+65
| | | | | | | | | | | | | | | | | | | | | | | | | | | | PHPBB3-11610