search

crownheightsaid / mutual-aid-app

Hey we're a mutual aid group in Crown Heights trying to help around town. Please reach out if you want to setup your own slack app and need a hand!
MIT License
25 stars 17 forks source link

Problem with twilio links in airtable for Firefox users #55

Open ericfg opened 4 years ago

ericfg commented 4 years ago

the urls that are pushed into the airtable with a link to the twilio message do not work in Firefox. the url needs to have .mp3 (or .wav) appended to it for it to work in all browsers. the current urls lead to an unusable text file being downloaded image

mjmaurer commented 4 years ago

While this is an issue with the repo, is Flatbush using Twilio functions? The twilio functionality in this repo is new and I think your Twilio code may exist soley in Twilio.

wrudebusch commented 4 years ago

@ericfg do you have a link I can try. I use FireFox on Ubuntu.

ericfg commented 4 years ago

here's an example (took me a bit of digging to find a message that could be shared without being a privacy issue )

the url as written to airtable is https://api.twilio.com/2010-04-01/Accounts/ACb5ca9f8797f5a4a72331b9635f845c36/Recordings/RE32f8a82227a8b27f922cb94d0541344f

that will play in browser in chrome (tested on mac, windows, ubuntu), safari (only tested on mac), but not in firefox (tested on mac, windows, ubuntu) unless .mp3 is appended

I have no idea if this bug still exists, we disabled our incoming line to allow for time to catch up on the thousands of outstanding requests we had. won't be opening things up again for another week or three.

wrudebusch commented 4 years ago

oh weird. i can confirm this bug. Looking into it.

wrudebusch commented 4 years ago

@ericfg I can't find anything about this on the googles.

can we check for an extension then, if not present append ".mp3" to the string as it is being saved to Airtable?

ericfg commented 4 years ago

that makes sense to me. thanks for digging into this