vmario/YOURLS-AuthMgrPlus
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Remove filter chain for role enumeration

Browse Source
This commit is contained in:
Nic Waller 2013-06-01 18:18:50 +00:00
parent 704a690c12
commit 35b79d89a3
1 changed files with 1 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
authmgr/plugin.php
View File

@ -18,7 +18,6 @@ if( !defined( 'YOURLS_ABSPATH' ) ) die();
* should be allowed to proceed. The filter names are defined here. * should be allowed to proceed. The filter names are defined here.
*/ */
define( 'AUTHMGR_ALLOW', 'filter_authmgr_allow' ); define( 'AUTHMGR_ALLOW', 'filter_authmgr_allow' );
define( 'AUTHMGR_HASROLE', 'filter_authmgr_hasrole' );
// Define constants used for naming roles (but they don't work in config.php) // Define constants used for naming roles (but they don't work in config.php)
class AuthmgrRoles { class AuthmgrRoles {
@ -279,20 +278,9 @@ function authmgr_check_apiuser_capability( $original, $capability ) {
/* /*
* Determine whether a specific user has a role. * Determine whether a specific user has a role.
* Currently based on role definitions in user/config.php
*/ */
function authmgr_user_has_role( $username, $rolename ) { function authmgr_user_has_role( $username, $rolename ) {
return yourls_apply_filter( AUTHMGR_HASROLE, false, $username, $rolename );
}
// ******************* FILTERS THAT GRANT ROLE MEMBERSHIP *********************
// By filtering AUTHMGR_HASROLE, you can connect internal roles to something else.
// Any filter handlers should execute as quickly as possible.
/*
* What role memberships are defined for the user in user/config.php?
*/
yourls_add_filter( AUTHMGR_HASROLE, 'authmgr_user_has_role_in_config');
function authmgr_user_has_role_in_config( $original, $username, $rolename ) {
global $authmgr_role_assignment; global $authmgr_role_assignment;
// if no role assignments are created, grant everything // if no role assignments are created, grant everything