Skip to content

Conversation

@binon
Copy link
Contributor

@binon binon commented Jul 3, 2025

*** PLEASE DO NOT OPEN PULL REQUESTS VIA GITHUB ***

The moodle.git repository at Github is just a mirror of the official repository. We do not accept pull requests at Github.

See CONTRIBUTING.md guidelines for how to contribute patches for Moodle. Thank you.

--

ColinBeebyHEE and others added 30 commits April 1, 2025 14:30
…es/TD-5136-Investigate-options-for-adding-course-substructure-and-resources-for-use-in-the-migration-tool-with-unit-test

TD-5136:custom plugin created
…es/TD-5485-Modify-the-migration-tool-add-SCORM-content-to-courses-using-the-new-endpoint-parameter-changes

TD-5485: Modify the migration tool add SCORM content to courses using  the new endpoint
…es/TD-create-plugin-to-manage-org-specific-configs

New plug in- TEL Organisation Specific Configurations
…he user trigger sign out from learning hub
…ent-LH-single-sign-on-to-the-test-PGVLE-Moodle-instance

TD-5537: New custom page has been created to kill user session when the user trigger sign out from learning hub
…7-Moodle-self-entrollable-courses-should-be-discoverable-in-lh
…es/TD-5377-Moodle-self-entrollable-courses-should-be-discoverable-in-lh

Develop/features/td 5377 Moodle self enrollable courses should be discoverable in LH
…es/TD-5485Modify-the-migration-tool-add-SCORM-content-to-courses-using-the-new-endpoin

Develop/features/td 5485 modify the migration tool add scorm content to courses using the new endpoin
Binon and others added 26 commits May 29, 2025 14:13
…es/TD-5377-Moodle-self-entrollable-courses-should-be-discoverable-in-lh

Develop/features/td 5377 Moodle self enrollable courses should be discoverable in LH
…es/TD-5377-Moodle-self-entrollable-courses-should-be-discoverable-in-lh

Masked the token field
…es/TD-5377-Moodle-self-entrollable-courses-should-be-discoverable-in-lh

When course details updated fire the external API
…es/TD-5377-Moodle-self-entrollable-courses-should-be-discoverable-in-lh

Fixing the failure when catalogue id is null
…es/TD-5377-Moodle-self-entrollable-courses-should-be-discoverable-in-lh

Added validation on self entrol check
…es/TD-5698-Moodle-courses-searchable-using-subsection-title

Courses can be searched using section name and resource name
…es/TD-5698-Moodle-courses-searchable-using-subsection-title

removing/hiding activity and resources are now triggered
@binon binon merged commit edd00ae into Implement_OpenID_Connect Jul 3, 2025
12 checks passed
@gitguardian
Copy link

gitguardian bot commented Jul 3, 2025

⚠️ GitGuardian has uncovered 2 secrets following the scan of your pull request.

Please consider investigating the findings and remediating the incidents. Failure to do so may lead to compromising the associated services or software components.

🔎 Detected hardcoded secrets in your pull request
GitGuardian id GitGuardian status Secret Commit Filename
15558685 Triggered Generic Password 1fb4843 auth/oidc/classes/loginflow/base.php View secret
5785368 Triggered Generic Password 1fb4843 auth/oidc/tests/privacy_provider_test.php View secret
🛠 Guidelines to remediate hardcoded secrets
  1. Understand the implications of revoking this secret by investigating where it is used in your code.
  2. Replace and store your secrets safely. Learn here the best practices.
  3. Revoke and rotate these secrets.
  4. If possible, rewrite git history. Rewriting git history is not a trivial act. You might completely break other contributing developers' workflow and you risk accidentally deleting legitimate data.

To avoid such incidents in the future consider


🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants