[mythtv] [mythtv-commits] mythtv branch master updated by stuartm. v0.28-pre-1486-g96bb040

[Stuart Morgan]

On Friday 06 Jun 2014 12:46:58 George Nassas wrote:
> > commit 3ab388c99b77e5b5b96ad267fc501ec5d732f5f5
> > Author:    Stuart Morgan <smorgan at mythtv.org> at Tue, 3 Jun 2014 23:10:41
> > +0100 Committer: Stuart Morgan <smorgan at mythtv.org> at Tue, 3 Jun 2014
> > 23:30:01 +0100 URL:      
> > http://code.mythtv.org/cgit/mythtv/commit/?id=3ab388c99b77e5b5b96ad267fc5
> > 01ec5d732f5f5
> > 
> > Services API: Fix a few endpoints which should have been POST only.
> > RFC 2616 states "In particular, the convention has been established
> > that the GET and HEAD methods SHOULD NOT have the significance of
> > taking an action other than retrieval. These methods ought to be
> > considered 'safe'."
> > 
> > In other words, any endpoint which alters data or which triggers any
> > action such as starting video playback, sending notifications etc
> > should use POST. These are things that should not be triggered by
> > accident.
> 
> Just curious, shouldn't these changes implied a class version change too? If
> you have a protocol connection open you can test for >= 82 but a
> services-only client is shooting in the dark.

I could have sworn I bumped the versions ... strange.

I'll do that now.

If anyone using the API had been paying attention to the documentation they 
would already be using POST as this 'rule' was outlined there.

-- 
Stuart Morgan