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

Orcid two factor authentication #32

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

JackBlackLight
Copy link

Thanks for your work on this plugin. We will be using it on our OJS instance with OrcId as our openid provider, but in addition, we are requiring our users to enable 2FA in their OrcId accounts. The code in this PR enforces the 2FA requirement and adds this option to the OrcId section of the settings form.

Also, I was able to get the plugin working without requiring the vendor autoloader.php file on line 3 of OpenIDHandler. I'm not sure if I’m missing something important there or if that is leftover from a previous approach.

-Jack

@RBoelter
Copy link

Without autoload.php and the dependent phpseclib lib, it is not possible to use OpenID providers with a cert modulus and exponent. You need to run composer install to create the vendor folder and load the dependencies. It might work with ORCID without autoload, but I am 100% sure that you have eliminated e.g. Keycloak as provider.

Copy link

@RBoelter RBoelter left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

change // $loader = require('plugins/generic/openid/vendor/autoload.php');
to $loader = require('plugins/generic/openid/vendor/autoload.php');
and run composer install in your terminal

@JackBlackLight
Copy link
Author

Thanks for your feedback. I've restored the autoloader requirement.

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.

None yet

2 participants