search

SAML-Toolkits / wordpress-saml

OneLogin SAML plugin for Wordpress
MIT License
65 stars 74 forks source link

Use the worpress roles API #19

Closed klondi closed 8 years ago

klondi commented 8 years ago

Use the worpress roles API to generate the options for the mappings and use these mappings to set the user role. For now the first found valid role is used.

pitbulk commented 8 years ago

@klondi Nice for that contribution, I will test it and publish a new version of the plugin that will include that improvements.

klondi commented 8 years ago

@pitbulk Found a bug where the default role would be set when no role mapping is set it's fixed with this new push.

Also it makes it possible to set role precedences (so in case a user is in many groups and thus many role attributes are send, the user can choose which role should be used).

pitbulk commented 8 years ago

I contacted some customers that are using this plugin and they will be able to check if this new feature don't break anything (they use custom roles on their wordpress site).

I will wait till they give their ok. (it will take 1-2 weeks).

klondi commented 8 years ago

@pitbulk no worries, I'll be working on this project still for a few weeks so if they find any issue I'll be able to fix it :)