← Back to team overview

mahara-contributors team mailing list archive

[Bug 1950199] [NEW] SimpleSamlphp metadata refresh causing cron problems

 

Public bug reported:

When we have:
1) a site has a timezone / country selected
2) a site that has SAML installed
3) an institution set up with a SAML auth instance
4) the SAML auth instance has 'Metadata URL for auto-refresh' set to a valid value

The timezone gets changed because of the getTime() method in the
htdocs/auth/saml/extlib/simplesamlphp/modules/metarefresh/lib/MetaLoader.php
file setting the timezone to UTC.

This means any cron job happening after the
PluginAuthSaml::auth_saml_refresh_cron one will be using the wrong
'time' to compare to and so some cron jobs won't run when they should.

** Affects: mahara
     Importance: High
         Status: New

** Affects: mahara/20.10
     Importance: High
         Status: New

** Affects: mahara/21.04
     Importance: High
         Status: New

** Affects: mahara/21.10
     Importance: High
         Status: New

** Affects: mahara/22.04
     Importance: High
         Status: New

** Also affects: mahara/21.10
   Importance: Undecided
       Status: New

** Also affects: mahara/20.10
   Importance: Undecided
       Status: New

** Also affects: mahara/22.04
   Importance: Undecided
       Status: New

** Also affects: mahara/21.04
   Importance: Undecided
       Status: New

** Changed in: mahara/22.04
   Importance: Undecided => High

** Changed in: mahara/21.10
   Importance: Undecided => High

** Changed in: mahara/21.04
   Importance: Undecided => High

** Changed in: mahara/20.10
   Importance: Undecided => High

** Changed in: mahara/22.04
    Milestone: None => 22.04.0

** Changed in: mahara/21.10
    Milestone: None => 21.10.1

** Changed in: mahara/21.04
    Milestone: None => 21.04.3

** Changed in: mahara/20.10
    Milestone: None => 20.10.4

-- 
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
Matching subscriptions: mahara-contributors
https://bugs.launchpad.net/bugs/1950199

Title:
  SimpleSamlphp metadata refresh causing cron problems

Status in Mahara:
  New
Status in Mahara 20.10 series:
  New
Status in Mahara 21.04 series:
  New
Status in Mahara 21.10 series:
  New
Status in Mahara 22.04 series:
  New

Bug description:
  When we have:
  1) a site has a timezone / country selected
  2) a site that has SAML installed
  3) an institution set up with a SAML auth instance
  4) the SAML auth instance has 'Metadata URL for auto-refresh' set to a valid value

  The timezone gets changed because of the getTime() method in the
  htdocs/auth/saml/extlib/simplesamlphp/modules/metarefresh/lib/MetaLoader.php
  file setting the timezone to UTC.

  This means any cron job happening after the
  PluginAuthSaml::auth_saml_refresh_cron one will be using the wrong
  'time' to compare to and so some cron jobs won't run when they should.

To manage notifications about this bug go to:
https://bugs.launchpad.net/mahara/+bug/1950199/+subscriptions



Follow ups