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

Document security implications when using the local connector #27

Open
jensschuppe opened this issue Jun 3, 2020 · 1 comment
Open

Comments

@jensschuppe
Copy link
Collaborator

The local connector (as opposed to remote connectors) does not require a Drupal user and thus no permissions to send CiviCRM API requests via the CiviMRF interface.

The permission to add/edit CiviMRF profiles should therefore be marked as a restricted permission (see the hook_permission() documentation) and a warning should be printed on the configuration page, explaining the implications involved with using the local connector.

A problematic scenario would be Webform editors being granted the permission to configure CiviMRF for Webforms, which allows selecting the CiviMRF profile, regardless of whether the local or a remote connector is being used.

@jensschuppe
Copy link
Collaborator Author

jensschuppe commented Sep 13, 2021

This involves introducing a dedicated permission, as the administer site configuration permission is currently being used for that, and it already has the "restricted" designation.

Also, in the D8/9 version of the module, the local connector can not be used anymore. This leaves this issue with missing documentation.

@jensschuppe jensschuppe removed this from the 2.0.0 milestone Sep 13, 2021
@jensschuppe jensschuppe changed the title Add warnings for permissions regarding local connector usage Document security implications when using the local connector Sep 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant