dgeller-ouhsc / winttsservice Goto Github PK

@dgeller-OUHSC (cc: @thomasnwilson ) After our initial look at the code, we have a few reactions below. An larger question is does Campus Policy require consulting Randy and April, since this solution involves more than just REDCap. Let's remember to discuss this during our afternoon meeting.

1. The text that is being converted to audio comes from the item prompt, correct? And the prompts sometimes pipe information from previous responses, correct? In that case, could PHI ever be included in the text that is converted to audio? How are you ensuring this could not happen? If it must happen, the larger architecture probably has to pass through April & Randy for approval.

2. We're concerned that foundational REDCap source code is being altered. Let's talk about alternative solutions.

   Unfortunately, REDCap hasn’t made the TTS url a setting yet, so in order to get REDCap to use the service, 1 line needs to be changed in speak.php

   • Does this change break any links or something with the rest of the source code? Even if it doesn't in the current version, it will need to be checked for every future upgrade. A ramification is that this project would need to be tightly coupled with the REDCap admin during upgrades, or else your users will be denied (at best).
   • How could this affect other users and projects?

3. How can convertText be more thoroughly sanitized?

   BTW, we see the urlencode() function on the REDCap side (in speak.php).