search

google-code-export / flowplayer-core

Automatically exported from code.google.com/p/flowplayer-core
2 stars 0 forks source link

Allow for support for backend email cgi scripts with viralvideos #618

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
See: http://code.google.com/p/flowplayer-core/issues/detail?id=401#c7

Due to these types of scripts not returning the required json format for user 
confirmation, no user feedback may be possible. Not sure of a "standard" one to 
use for testing, I only know of many free scripts that were being used back in 
the 90's. Mostly all require some return url or output html.

Original issue reported on code.google.com by electrot...@gmail.com on 7 Aug 2012 at 1:51

GoogleCodeExporter commented 9 years ago 
This is the one preinstalled on our server:
http://web.mit.edu/wwwdev/cgiemail/
Most of those are pretty old, yes.

As far as I understand it many of them leave this to sendmail. Also spam 
prevention etc. is up to the cgi script, not something we have to provide.

Original comment by blacktrashproduct on 7 Aug 2012 at 4:39

GoogleCodeExporter commented 9 years ago 
Please try this 
https://dl.dropbox.com/u/3394987/flowplayer.viralvideos-3.2.11.zip

From the email, blank output from these scripts is treated as success, it will 
then try and parse json output from messages to be returned if that fails 
because it's html it will also return success, but log differently with the 
output also on the log. This helps debug things also for sending problems

Original comment by electrot...@gmail.com on 8 Aug 2012 at 6:05

GoogleCodeExporter commented 9 years ago 
So the issue was sending actually works but because the json parser fails 
because of the html output it traps this and sends an error message which I had 
changed recently to show success, I'm not sure if you got that version as it 
was committed already this should improve that.

Original comment by electrot...@gmail.com on 8 Aug 2012 at 6:07

GoogleCodeExporter commented 9 years ago 
confirm this is all working now ?

Original comment by electrot...@gmail.com on 8 Aug 2012 at 5:14

GoogleCodeExporter commented 9 years ago 
Yup. Closing.

Original comment by blacktrashproduct on 8 Aug 2012 at 6:48

GoogleCodeExporter commented 9 years ago 
reopening to remove reundant code

Original comment by electrot...@gmail.com on 14 Aug 2012 at 12:59

GoogleCodeExporter commented 9 years ago

Original comment by electrot...@gmail.com on 14 Aug 2012 at 1:52