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 683287 - gst-rtsp-server: add rtsp requests signals to rtsp-client
gst-rtsp-server: add rtsp requests signals to rtsp-client
Status: RESOLVED FIXED
Product: GStreamer
Classification: Platform
Component: gst-rtsp-server
0.10.8
Other Linux
: Normal normal
: 0.10.9
Assigned To: GStreamer Maintainers
GStreamer Maintainers
Depends on:
Blocks:
 
 
Reported: 2012-09-03 17:45 UTC by Aleix Conchillo Flaqué
Modified: 2012-09-07 16:20 UTC
See Also:
GNOME target: ---
GNOME version: ---

Attachments
add rtsp request signals to client (7.21 KB, patch)
2012-09-03 18:20 UTC, Aleix Conchillo Flaqué 		none Details | Review

Description Aleix Conchillo Flaqué 2012-09-03 17:45:29 UTC 
Sometimes, it is a bit hard to track down client, media, session, etc. relationship. For example, when a client connects to the server we can connect to the "client-connected" signal, but at that point we still don't know which URI has the client connected to.

It would be good to have additional signals for each client RTSP request.
Comment 1 Aleix Conchillo Flaqué 2012-09-03 18:20:38 UTC 
Created attachment 223356 [details] [review]
add rtsp request signals to client

This patch emits signals after each RTSP request is handled. The "teardown-request" signal is emitted before the client connection is closed.
Comment 2 Wim Taymans 2012-09-07 14:42:19 UTC 
commit 3b2498d9cbf023f849f1b5f0cce013e9a21d181c
Author: Aleix Conchillo Flaque <aleix@oblong.com>
Date:   Mon Sep 3 10:48:14 2012 -0700

    rtsp-client: add signals for rtsp requests (fixes #683287)
Comment 3 Tim-Philipp Müller 2012-09-07 16:16:01 UTC 
Aleix: would be great if you could put the bug numbers into the commit message body in future, for consistency with the other commit messages (and because it's not useful information when quickly scanning the summary lines for changes), as per http://gstreamer.freedesktop.org/wiki/SubmittingPatches
Comment 4 Aleix Conchillo Flaqué 2012-09-07 16:20:57 UTC 
OK, got it. Thanks for the heads up!