search

vanderbilt-redcap / email-alerts-module

Module to send customized emails alerts after a form/survey is completed to one or several recipients.
MIT License
3 stars 9 forks source link

Redcap variables in attachment leads to random attachments of other projects #9

Closed mikestoller closed 6 years ago

mikestoller commented 6 years ago

REDCap 8.1.8 LTS. Redcap variables in attachment lead to Email Alert containing random attachments of other projects, e.g. pdfs and reports of other projects.

knil-maloon commented 6 years ago

Hi @mikestoller is the project repeatable? If so the event needs to be specified as well as the variable name. As for other projects, do you mean instruments? Email alerts is activated by project so it sounds very weird for it going to other projects. Other instruments is normal as long as the variable name is the same. You can also you can check in the "Logging" section which alerts were activated on which records and the email that was sent. That might be helpful to see what might be going on.

mikestoller commented 6 years ago

@knil-maloon the project is non-repeatable and non-longitudinal. However, the error might also be due to an incomplete "dump" on the test version of redcap where I tested the module.

knil-maloon commented 6 years ago

@mikestoller I just made a test of an attached variable in the test server and it send me the email with the attached file I uploaded

mikestoller commented 6 years ago

@knil-maloon Thank you. To clarify: I put variables into the attachment, which were not of field type "file upload". I then got the random attachments.

knil-maloon commented 6 years ago

@mikestoller Oh! Well the variable attachment only works for file types if you try to attach something that is not a document but a normal variable I guess it's normal it does crazy things haha.

mikestoller commented 6 years ago

@knil-maloon ah, I see xD Well, the problem is that this way people can get confidential data from other projects, so you might still want to consider to address it ;)

knil-maloon commented 6 years ago

Yeah, it's a good point. Thank you for your suggestion, we will take it under consideration for future updates.

knil-maloon commented 6 years ago

Fixed on version 1.3.7