Siegfried Wagner
2005-Jan-17 01:25 UTC
[Icecast-dev] patch for icecast-2.2.0 to add client maxtime
Hi, I wrote a little patch for the stable version (also works for the svn version) to add a new configuration parameter called "client_maxtime". With this you can set a maximum connection time limit for a connected client so that you can disallow continuous listening. When the listening time exceeds the client connection will automatically be dropped. By default this feature is disabled (set to 0). -------------- next part -------------- A non-text attachment was scrubbed... Name: icecast-2.2.0-client_maxtime.diff Type: application/octet-stream Size: 6958 bytes Desc: not available Url : http://lists.xiph.org/pipermail/icecast-dev/attachments/20050117/846d0110/icecast-2.2.0-client_maxtime.obj
Michael Smith
2005-Jan-17 14:51 UTC
[Icecast-dev] patch for icecast-2.2.0 to add client maxtime
On Mon, 17 Jan 2005 10:25:32 +0100, Siegfried Wagner <siggi@radiofreaks.org> wrote:> Hi, > > I wrote a little patch for the stable version (also works for the svn > version) to add a new configuration parameter called "client_maxtime". > With this you can set a maximum connection time limit for a connected > client so that you can disallow continuous listening. > When the listening time exceeds the client connection will > automatically be dropped. > By default this feature is disabled (set to 0).This looks like a potentially useful feature. There are a number of problems in your patch, though: - unsigned/signed confusion. client_maxtime should be unsigned everywhere, not just in some places. - the check for whether to disconnect should probably be in send_to_listener, rather than directly cluttering source_main. - The formatting is messed up, inconsistent with the style of the rest of the code, and has several excessively long lines. If you could clean up those issues and send us another patch, then (assuming others agree that this looks useful), we'd be happy to include it. Mike
Greg J. Ogonowski
2005-Jan-17 15:07 UTC
[Icecast-dev] patch for icecast-2.2.0 to add client maxtime
Definitely agree this is a useful feature. -greg. At 14:50 2005-01-17, Michael Smith wrote:>On Mon, 17 Jan 2005 10:25:32 +0100, Siegfried Wagner ><siggi@radiofreaks.org> wrote: > > Hi, > > > > I wrote a little patch for the stable version (also works for the svn > > version) to add a new configuration parameter called "client_maxtime". > > With this you can set a maximum connection time limit for a connected > > client so that you can disallow continuous listening. > > When the listening time exceeds the client connection will > > automatically be dropped. > > By default this feature is disabled (set to 0). > >This looks like a potentially useful feature. There are a number of >problems in your patch, though: > - unsigned/signed confusion. client_maxtime should be unsigned >everywhere, not just in some places. > - the check for whether to disconnect should probably be in >send_to_listener, rather than directly cluttering source_main. > - The formatting is messed up, inconsistent with the style of the >rest of the code, and has several excessively long lines. > >If you could clean up those issues and send us another patch, then >(assuming others agree that this looks useful), we'd be happy to >include it. > >Mike >_______________________________________________ >Icecast-dev mailing list >Icecast-dev@xiph.org >http://lists.xiph.org/mailman/listinfo/icecast-dev__________________________________________________________________________ Greg J. Ogonowski VP Product Development ORBAN / CRL, Inc. 1525 Alvarado St. San Leandro, CA 94577 USA TEL +1 510 351-3500 FAX +1 510 351-0500 greg@orban.com http://www.orban.com