Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

AddRecordingLiveStream Fails With 0.27 Backend. UTC Issue #37

Closed
billmeek opened this issue Apr 9, 2015 · 4 comments
Closed

AddRecordingLiveStream Fails With 0.27 Backend. UTC Issue #37

billmeek opened this issue Apr 9, 2015 · 4 comments
Assignees
Labels

Comments

@billmeek
Copy link
Contributor

billmeek commented Apr 9, 2015

The app's requested Start Time (without URL encoding is):

StartTime=2015-04-08T20:33:00.000-05:00

The failure in mythbackend.log is:

AddRecordingLiveStream - for 1091_2015-04-08T20:33:00Z  failed

The file in the BE is named:

1091_20150409013300.mpg

It appears that 0.28-pre gets the same information, however, the
recordedid is used and the chanid/starttime are ignored.

04-08 20:55:27.156  19760-19786/org.mythtv.android D/Retrofit﹕ ---> HTTP GET http://192.168.1.220:6544/Content/AddRecordingLiveStream?ChanId=1091&StartTime=2015-04-08T20%3A33%3A00.000-05%3A00&MaxSegments=0&Width=1280&Height=720
04-08 20:55:27.156  19760-19786/org.mythtv.android D/Retrofit﹕ Accept: application/json
04-08 20:55:27.156  19760-19786/org.mythtv.android D/Retrofit﹕ MYTH_REQUEST_ID: LIVE_STREAM_REQ_ID
04-08 20:55:27.156  19760-19786/org.mythtv.android D/Retrofit﹕ ---> END HTTP (no body)
04-08 20:55:27.186  19760-19786/org.mythtv.android D/Retrofit﹕ <--- HTTP 404 http://192.168.1.220:6544/Content/AddRecordingLiveStream?ChanId=1091&StartTime=2015-04-08T20%3A33%3A00.000-05%3A00&MaxSegments=0&Width=1280&Height=720 (28ms)
04-08 20:55:27.186  19760-19786/org.mythtv.android D/Retrofit﹕ Server: Linux 3.13.0-44-generic, UPnP/1.0, MythTV 0.27.20141016-1
04-08 20:55:27.196  19760-19786/org.mythtv.android D/Retrofit﹕ <?xml version="1.0" encoding="utf-8"?><detail><errorCode>501</errorCode><errorDescription>Call to method failed</errorDescription></detail><?xml version="1.0" encoding="utf-8"?><detail><errorCode>401</errorCode><errorDescription>Invalid Action</errorDescription></detail>
@dmfrey
Copy link
Contributor

dmfrey commented Apr 9, 2015

This is a problem in the mythtv-services-api. It is not serializing the data correctly before sending it.

@tafypz where in the api can we override setting the date format for outbound calls. I see there is already a DateTimeSerializer in the code and it annotates the DateTime fields when when the json is read into the objects. That, unfortunately, has the wrong format. Any ideas? Does your generator need to be updated?

@billmeek billmeek assigned tafypz and unassigned dmfrey Apr 9, 2015
@billmeek billmeek changed the title AddRecordingLiveStream Fails With 0.27 Backend. UTF Issue AddRecordingLiveStream Fails With 0.27 Backend. UTC Issue Apr 9, 2015
@dmfrey dmfrey assigned dmfrey and unassigned tafypz Apr 9, 2015
@dmfrey
Copy link
Contributor

dmfrey commented Apr 9, 2015

@billmeek I think i can relieve this by changing the the calls on v027 to AddLiveStream from AddRecordingLiveStream. I am going to commit this. Can you test it out? I also wonder if this won't resolve Issue #3 since i am passing in a storage group and hostname. If you can test that out on v27 as well and it works, I could make this a permanent change for all backends.

@billmeek
Copy link
Contributor Author

billmeek commented Apr 9, 2015

Yes, I'll test the 0.27 StartTime AddLiveStream workaround.

I can't test for #3 in the 0.27 version (I've only got it running on 1 host.)
But I can test it on 0.28-pre. Maybe just using curl as a proof of concept
would be a good idea. It should fail because the MBE won't find the file
(even though it's in the Storage Group, the Storage Directory doesn't
exist on the MBE, e.g. it's not cross mounted.)

@dmfrey
Copy link
Contributor

dmfrey commented Apr 9, 2015

@billmeek I can leave this change in for all backends. I am testing it now on v28 and it works. Maybe I will leave that in like this for now. See how it works.

@dmfrey dmfrey closed this as completed in 8177f1f Apr 9, 2015
dmfrey pushed a commit that referenced this issue Apr 9, 2015
dmfrey pushed a commit that referenced this issue Apr 9, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants