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

Move to redcap domain verification only and avoiding subsequent issues #67

Open
yatharthranjan opened this issue May 3, 2023 · 3 comments

Comments

@yatharthranjan
Copy link
Member

yatharthranjan commented May 3, 2023

Currently, we have an issue with the amount of manual work required with updating the redcap urls in the config whenever the redcap versions are updated as the urls contain redcap version numbers (like https://redcap-phidatalab.brc.iop.kcl.ac.uk/redcap_v13.1.27/). This is not scalable and hence we need to move to using the base domain name verification and the redcap version can be anything in that case. But this causes issues if there are more than one redcap is hosted on a single domain. This thread suggests mitigations for that.

We should verify that the token provided can actually access redcap form before we generate a subject on management portal to avoid inconsistencies since later the app may not be able to update the redcap form if the token is not valid for the redcap project being updated. Ideally, this should be enabled (disabled by default) with a property like preverify_redcap_access: true since we do have some studies where redcap is not accessible and they are happy with a one way integration with management portal.
I think this would be quite useful when we move to validation of url to just use a domain name instead of full url, as it would allow us to avoid incorrect use of redcap + mp mapping if more than one redcap is hosted on a single domain as the token for projects would ideally be different and we will not be able to access redcap if it is incorrectly mapped.
What do you think @afolarin and @mpgxvii ?

@yatharthranjan
Copy link
Member Author

Additionally, we can also add a key to map the redcap versions on a single deployment, allowing integration for mutiple redcaps from a single domain. If this key is not specified, it would fail as mentioned above.

@afolarin
Copy link
Member

afolarin commented May 9, 2023

Potential options?:

  • Use API key to check project access to disambiguate projects on the TLDN (although this wouldn't work if you had multiple versions of a project on multiple versions of REDCap deployed on different version URL paths).
  • Have a Key or Code in the radar-integration form that could be used to disambiguate projects on the TLDN.

@yatharthranjan
Copy link
Member Author

Probably redundant now with the new SEP which does not use Redcap anymore

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