[mythtv] [mythtv-commits] mythtv/master commit: 59f88dd88 by Robert McNamara (rmcnamara)

Michael T. Dean mtdean at thirdcontact.com
Thu Apr 7 03:12:19 UTC 2011


On 04/06/2011 10:42 PM, David Blain wrote:
>> API: Add a SendMessage method to the Myth service.
>>
>> More or less exactly approximates the functionality of the mythmessage
>> program, but is not usable from a browser, scriptable, callable from JSON,
>> etc.  Causes a message to appear on all frontends, or on a single frontend.
>> Usage:
>>
>> http://BackendServerIP:6544/Myth/SendMessage?Message=Here is some
>> message text&Address=192.168.0.5&udpPort=6948
>>
>> Address and udpPort arguments are both optional. They default to
>> 255.255.255.255 (all connected frontends) and 6948, respectively. So send a message to all frontends at once, quickly:
>>
>> http://BackendServerIP:6544/Myth/SendMessage?Message=The system is
>> going down!
>>
>> Modified:
>>
>>     mythtv/libs/libmythservicecontracts/services/mythServices.h
>>     mythtv/programs/mythbackend/services/myth.cpp
>>     mythtv/programs/mythbackend/services/myth.h
>>
> Since this call technically sets/sends a value, I think it would be better
> if it was limited to POST calls only.
>
> Just my $0.02...

It's not really modifying server-side state, or even saving data for 
that matter.  And allowing GET makes it a lot easier to access, so I'd 
think it should be fine.  Definitely won't cause accidental 
irrecoverable loss of data.

Mike


More information about the mythtv-dev mailing list