search

microsoft / azure-redcap-paas

Automated deployment of REDCap with Azure Blob storage as the storage back-end
MIT License
28 stars 55 forks source link

Deploy # 2 #9

Closed pauldotyu closed 3 years ago

pauldotyu commented 3 years ago

This is the correct pull request to add additional checks on the $header and $filename in the GetFileName function

taylorr4 commented 3 years ago

@bretthackermsft do you have any thoughts on this? Look alright to you?

bretthackermsft commented 3 years ago

Interesting - I suppose we're dealing with the case of empty strings populating those vars. Not sure what could cause that but I don't see any harm in the additional test.

taylorr4 commented 3 years ago

@bretthackermsft thanks for your input on this. If it sounds okay to you, then i will go ahead and merge these changes then.