Non-core contributed modules

SSO Auth module for SAML 2.0 based login using the SimpleSAMLPHP libraries

Details

  • Type: New Feature New Feature
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 1.9.4
  • Fix Version/s: None
  • Component/s: Auth: SAML
  • Labels:
    None
  • Database:
    Any
  • Affected Branches:
    MOODLE_19_STABLE

Description

A new Moodle Auth module to provide SAML 2.0 SSO integration based on the Simple SAML PHP libraries (http://rnd.feide.no/simplesamlphp). This is a request to help organise the submission of a new module based on work that has been started at http://code.google.com/p/simplesamlphp-moodle/. So far I have worked through this code, doing some tidying up, improving the configuration handling, and the integration with Moodle sessions.
Currently the module is named auth/feide, but based on discussion in the simplesamlphp mailing list (http://groups.google.com/group/simplesamlphp/browse_thread/thread/ddadc49f06b8b4ad) it is agreed that this should be changed, and and was proposed that it should be auth/saml.
If possible, can this name change be agreed first, and then I will align the code appropriately with this, and upload it for review?

Thanks,
Piers Harding.

Activity

Hide
Anthony Borrow added a comment -

Piers - Thanks for sharing the saml authentication plugin. It is best that we get the name agreed upon first before uploading the code to CVS. I say this only because of how CVS handles (or perhaps more accurately does not handle) the deletion or renaming of files. I would rather see us get the files properly named and then add it. Let me know when you feel it is ready and I will review it for possible inclusion into CVS. Don't hesitate if there is anything I can do to be helpful. Peace - Anthony

Show
Anthony Borrow added a comment - Piers - Thanks for sharing the saml authentication plugin. It is best that we get the name agreed upon first before uploading the code to CVS. I say this only because of how CVS handles (or perhaps more accurately does not handle) the deletion or renaming of files. I would rather see us get the files properly named and then add it. Let me know when you feel it is ready and I will review it for possible inclusion into CVS. Don't hesitate if there is anything I can do to be helpful. Peace - Anthony
Hide
Piers Harding added a comment -

Hi Anthony -

Thanks.

For now, I've named the module auth/saml. I'll upload it now for review.

Cheers,
Piers Harding.

Show
Piers Harding added a comment - Hi Anthony - Thanks. For now, I've named the module auth/saml. I'll upload it now for review. Cheers, Piers Harding.
Hide
Piers Harding added a comment -

First version of auth/saml prepared for inclusion in contrib.

Show
Piers Harding added a comment - First version of auth/saml prepared for inclusion in contrib.
Hide
Anthony Borrow added a comment -

moving to newly created Auth: SAML component

Show
Anthony Borrow added a comment - moving to newly created Auth: SAML component
Hide
Anthony Borrow added a comment -

Piers - I have added the SAML authentication plugin to CONTRIB, created a component in the tracker and assigned you as the component lead. You already have access to CONTRIB so should be able to maintain it without problem. Let me know if you need a separate branch (i.e. one for HEAD and another for 1.9) as I currently just added the code to HEAD. Peace - Anthony

Show
Anthony Borrow added a comment - Piers - I have added the SAML authentication plugin to CONTRIB, created a component in the tracker and assigned you as the component lead. You already have access to CONTRIB so should be able to maintain it without problem. Let me know if you need a separate branch (i.e. one for HEAD and another for 1.9) as I currently just added the code to HEAD. Peace - Anthony
Hide
Piers Harding added a comment -

Thanks a lot for sorting this out - I think I will need a branch for 1.9 too, if possible.

Cheers,
Piers Harding.

Show
Piers Harding added a comment - Thanks a lot for sorting this out - I think I will need a branch for 1.9 too, if possible. Cheers, Piers Harding.
Hide
Anthony Borrow added a comment -

Despite being an authentication plugin, this is really not a security related issue. If there are bugs in it then those could be security issues. Peace - Anthony

Show
Anthony Borrow added a comment - Despite being an authentication plugin, this is really not a security related issue. If there are bugs in it then those could be security issues. Peace - Anthony
Hide
Anthony Borrow added a comment -

Piers - I have added a 1.9 branch as well. Peace - Anthony

Show
Anthony Borrow added a comment - Piers - I have added a 1.9 branch as well. Peace - Anthony
Hide
Anthony Borrow added a comment -

Closing all of my resolved issues. Peace - Anthony

Show
Anthony Borrow added a comment - Closing all of my resolved issues. Peace - Anthony

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: