Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Totara 18 now includes paragonIE used in totp factor. #437

Closed
danmarsden opened this issue Sep 20, 2023 · 4 comments
Closed

Totara 18 now includes paragonIE used in totp factor. #437

danmarsden opened this issue Sep 20, 2023 · 4 comments

Comments

@danmarsden
Copy link
Member

Totara 18 (TL-37451) added the paragonIE constant_time_encoding stuff which now conflicts with the totp factor - we probably need to create a new TOTARA_18 branch and remove the external lib.

@aneno-m-e
Copy link

I just pushed a new TOTARA_18_STABLE branch.

  • checked out from the TOTARA_12 branch
  • added a commit to switch to using the core library for ParagonIE

@danmarsden
Copy link
Member Author

Thanks @aneno-m-e I've just renamed that branch to "TOTARA_18" - removing the stable text and dropped a patch into the readme on all branches mentioning the new branch.

Do you know if the patches mentioned in the readme are still needed on the Totara_18 branch? - also might be worth checking the requires version in teh version.php file to see if we can bump it higher to make it clear Totara 18 is required?

thanks!

@aneno-m-e
Copy link

I've updated the require version as the library does not exist in earlier versions of Tōtara.

The patches are still required . If #373 is merged, it could be used on this branch as well. A watcher has not been added to Tōtara core yet.

@danmarsden
Copy link
Member Author

Thanks!

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

No branches or pull requests

2 participants