After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 506025 - RTSPsrc doesn't handle redirects.
RTSPsrc doesn't handle redirects.
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-plugins-good
git master
Other All
: Normal normal
: 0.10.7
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2007-12-28 09:15 UTC by Vladimir Eremeev
Modified: 2007-12-31 13:28 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Patch to gstreamer plugins allowing handling redirects (301 and 302) (4.13 KB, patch)
2007-12-28 16:46 UTC, Vladimir Eremeev
rejected Details | Review
patch for handling redirects (4.98 KB, patch)
2007-12-29 13:51 UTC, Vladimir Eremeev
committed Details | Review

Description Vladimir Eremeev 2007-12-28 09:15:55 UTC
Please describe the problem:
When rtsp server replies with 301 or 302 (moved permanently or temporarily), rtspsrc treats this as error and stops.

Steps to reproduce:
1. Try to open any RTSP server, which redirects its clients.


Actual results:
gst-launch-0.10 rtspsrc location="rtsp://address.of.server/path/" ! fakesink
writes "got error 302 (Moved temporarily)", and stops

Expected results:
RTSPsrc was to get new location from Location: header and go to it.

Does this happen every time?
Yes.

Other information:
Comment 1 Vladimir Eremeev 2007-12-28 16:46:55 UTC
Created attachment 101742 [details] [review]
Patch to gstreamer plugins allowing handling redirects (301 and 302)
Comment 2 Vladimir Eremeev 2007-12-28 16:48:37 UTC
Sorry for inconveniences, probably paths to files should be fixed.
Comment 3 Vladimir Eremeev 2007-12-29 13:51:55 UTC
Created attachment 101786 [details] [review]
patch for handling redirects
Comment 4 Wim Taymans 2007-12-31 13:28:17 UTC
I modified your patch a bit, the redirect should change the status of the receiver and could happen with every request.

        * gst/rtsp/gstrtspsrc.c: (gst_rtspsrc_send), (gst_rtspsrc_open):
        * gst/rtsp/gstrtspsrc.h:
        Implement redirect for the DESCRIBE reply. Fixes #506025.