libapache2-mod-musicindex: generates playlist.m3u files having URI like http://somehost:0/blah

Automatically imported from Debian bug report #271912 http://bugs.debian.org/271912

Message-Id: <email address hidden>
Date: Wed, 15 Sep 2004 20:13:49 -0700
From: "Andrew D. Clark" <email address hidden>
To: Debian Bug Tracking System <email address hidden>
Subject: libapache2-mod-musicindex: generates playlist.m3u files having URI like
 http://somehost:0/blah

Package: libapache2-mod-musicindex
Version: 0.99.4
Severity: grave
Justification: renders package unusable

playlist files have URI specifying port 0 for http transport, resulting
in connection failure.

-- System Information:
Debian Release: 3.1
  APT prefers unstable
  APT policy: (500, 'unstable')
Architecture: i386 (i686)
Kernel: Linux 2.6.8
Locale: LANG=en_US, LC_CTYPE=en_US

Versions of packages libapache2-mod-musicindex depends on:
ii apache2-common 2.0.50-12 Next generation, scalable, extenda
ii libc6 2.3.2.ds1-16 GNU C Library: Shared libraries an
ii libflac4 1.1.0-11 Free Lossless Audio Codec - runtim
ii libid3tag0 0.15.1b-3 ID3 tag reading library from the M
ii libmad0 0.15.1b-1 MPEG audio decoder library
ii libvorbis0a 1.0.1-1 The Vorbis General Audio Compressi
ii libvorbisfile3 1.0.1-1 The Vorbis General Audio Compressi
ii mod-musicindex-common 0.99.4 Common files for mod-musicindex

-- no debconf information

Message-Id: <email address hidden>
Date: Sat, 18 Sep 2004 01:51:18 +0200
From: Thibaut VARENE <email address hidden>
To: "Andrew D. Clark" <email address hidden>
Cc: <email address hidden>
Subject: Re: mod_musicindex

tags 271912 confirmed
thanks

On Fri, 17 Sep 2004 16:45:57 -0700
"Andrew D. Clark" <email address hidden> wrote:
>
> Sorry, I'll try to be more clear :-)
>
> Since the URI printed to the playlist.m3u file specifies port 0 for
> http transport, connections fail (since my http is listening on port
> 80, not on port 0).

*G*

Seems that this actually comes from the previous patch that was sent to
fix the module behaviour with ssl servers.

As a matter of fact, it was only tested against Apache 1.3, for which it
works just fine. I've been able to reproduce your bug, and I am working
on a fix right now.

Thibaut VARENE
The PA/Linux ESIEE Team
http://www.pateam.org/

Message-Id: <email address hidden>
Date: Sat, 18 Sep 2004 02:16:17 +0200
From: Thibaut VARENE <email address hidden>
To: <email address hidden>
Cc: <email address hidden>, <email address hidden>,
 <email address hidden>
Subject: Re: Bug#271912: apache2 does not set request_rec->server->port

reassign 271912 apache2
tags 271912 + help
retitle 271912 apache2 does not set request_rec->server->port
thanks

Thibaut VARENE <email address hidden> wrote:

> tags 271912 confirmed
> thanks
>
> On Fri, 17 Sep 2004 16:45:57 -0700
> "Andrew D. Clark" <email address hidden> wrote:
> >
> > Sorry, I'll try to be more clear :-)
> >
> > Since the URI printed to the playlist.m3u file specifies port 0 for
> > http transport, connections fail (since my http is listening on port
> >
> > 80, not on port 0).
>

Replying to myself:

I tried downgrade to 0.99.2, it didn't fix the problem. It looks like a
bug in apache2, which no longer sets request_rec->server->port, making
the following piece of code both fail and misbehave:

        /* add the port number if needed */
        if (!ap_is_default_port(r->server->port, r)) {
                sprintf(str_port, ":%u", r->server->port);
                strcat(prefix, str_port);
        }

I'm therefore asking for help from the apache2 team and reassigning bug
to apache2.

HTH,

Thibaut VARENE
The PA/Linux ESIEE Team
http://www.pateam.org/

universe package

Message-ID: <004c01c49d25$6d5dc870$0400000a@0c3.net>
Date: Sat, 18 Sep 2004 12:15:51 +1000
From: "Adam Conrad" <adconrad@0c3.net>
To: "'Thibaut VARENE'" <email address hidden>
Cc: <email address hidden>
Subject: RE: Bug#271912: apache2 does not set request_rec->server->port

reassign 271912 libapache2-mod-musicindex
thanks

Thibaut VARENE wrote:
>=20
> I tried downgrade to 0.99.2, it didn't fix the problem. It looks like
a
> bug in apache2, which no longer sets request_rec->server->port, making
> the following piece of code both fail and misbehave:

No, if there is no "Port" directive in the conf, r->server->port is set
to 0, libapache-mod-musicindex should be using ap_get_server_port()
instead of the r->server struct. Note that this should work in both 1.3
and 2.0.

... Adam

Message-Id: <email address hidden>
Date: Sat, 18 Sep 2004 12:15:20 +0200
From: Thibaut VARENE <email address hidden>
To: "Adam Conrad" <adconrad@0c3.net>
Cc: <email address hidden>
Subject: Re: Bug#271912: apache2 does not set request_rec->server->port

tags 271912 - help
tags 271912 + pending
thanks

On Sat, 18 Sep 2004 12:15:51 +1000
"Adam Conrad" <adconrad@0c3.net> wrote:

> reassign 271912 libapache2-mod-musicindex
> thanks
>
> Thibaut VARENE wrote:
> >
> > I tried downgrade to 0.99.2, it didn't fix the problem. It looks
> > like
> a
> > bug in apache2, which no longer sets request_rec->server->port,
> > making the following piece of code both fail and misbehave:
>
> No, if there is no "Port" directive in the conf, r->server->port is
> set to 0, libapache-mod-musicindex should be using
> ap_get_server_port() instead of the r->server struct. Note that this
> should work in both 1.3 and 2.0.
>
> ... Adam

Thanks for the help, it fixes the bug!

Thibaut VARENE
The PA/Linux ESIEE Team
http://www.pateam.org/

Message-Id: <email address hidden>
Date: Sat, 18 Sep 2004 12:40:39 +0200
From: Thibaut VARENE <email address hidden>
To: <email address hidden>
Subject: Fw: Re: Bug#271912: apache2 does not set request_rec->server->port

tags 271912 - help
tags 271912 + pending
thanks

On Sat, 18 Sep 2004 12:15:51 +1000
"Adam Conrad" <adconrad@0c3.net> wrote:

> reassign 271912 libapache2-mod-musicindex
> thanks
>
> Thibaut VARENE wrote:
> >
> > I tried downgrade to 0.99.2, it didn't fix the problem. It looks
> > like
> a
> > bug in apache2, which no longer sets request_rec->server->port,
> > making the following piece of code both fail and misbehave:
>
> No, if there is no "Port" directive in the conf, r->server->port is
> set to 0, libapache-mod-musicindex should be using
> ap_get_server_port() instead of the r->server struct. Note that this
> should work in both 1.3 and 2.0.
>
> ... Adam

Thanks for the help, it fixes the bug!

Thibaut VARENE
The PA/Linux ESIEE Team
http://www.pateam.org/

Message-Id: <email address hidden>
Date: Sat, 18 Sep 2004 07:17:08 -0400
From: Thibaut VARENE <email address hidden>
To: <email address hidden>
Subject: Bug#271912: fixed in libapache-mod-musicindex 0.99.5

Source: libapache-mod-musicindex
Source-Version: 0.99.5

We believe that the bug you reported is fixed in the latest version of
libapache-mod-musicindex, which is due to be installed in the Debian FTP archive:

libapache-mod-musicindex_0.99.5.dsc
  to pool/main/liba/libapache-mod-musicindex/libapache-mod-musicindex_0.99.5.dsc
libapache-mod-musicindex_0.99.5.tar.gz
  to pool/main/liba/libapache-mod-musicindex/libapache-mod-musicindex_0.99.5.tar.gz
libapache-mod-musicindex_0.99.5_powerpc.deb
  to pool/main/liba/libapache-mod-musicindex/libapache-mod-musicindex_0.99.5_powerpc.deb
libapache2-mod-musicindex_0.99.5_powerpc.deb
  to pool/main/liba/libapache-mod-musicindex/libapache2-mod-musicindex_0.99.5_powerpc.deb
mod-musicindex-common_0.99.5_all.deb
  to pool/main/liba/libapache-mod-musicindex/mod-musicindex-common_0.99.5_all.deb

A summary of the changes between this version and the previous one is
attached.

Thank you for reporting the bug, which will now be closed. If you
have further comments please address them to <email address hidden>,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
Thibaut VARENE <email address hidden> (supplier of updated libapache-mod-musicindex package)

(This message was generated automatically at their request; if you
believe that there is a problem with it please contact the archive
administrators by mailing <email address hidden>)

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Format: 1.7
Date: Sat, 18 Sep 2004 12:17:15 +0200
Source: libapache-mod-musicindex
Binary: libapache-mod-musicindex libapache2-mod-musicindex mod-musicindex-common
Architecture: source powerpc all
Version: 0.99.5
Distribution: unstable
Urgency: high
Maintainer: Thibaut VARENE <email address hidden>
Changed-By: Thibaut VARENE <email address hidden>
Description:
 libapache-mod-musicindex - Browse, stream, download and search through MP3/Ogg files
 libapache2-mod-musicindex - Browse, stream, download and search through MP3/Ogg files
 mod-musicindex-common - Common files for mod-musicindex
Closes: 271912
Changes:
 libapache-mod-musicindex (0.99.5) unstable; urgency=high
 .
   * Fixes serious breakage when apache conf doesn't have Port directive
     (closes: #271912)
Files:
 27b1afbdb2fb6af40a31f23184e1b552 712 web optional libapache-mod-musicindex_0.99.5.dsc
 542c13bebf90743030a18f2c45e6e819 76759 web optional libapache-mod-musicindex_0.99.5.tar.gz
 b61ea985510f9939035750bafbbc618b 19546 web optional mod-musicindex-common_0.99.5_all.deb
 96ec9f42fe70d4b5e9e318c2c3ca4b26 28700 web optional libapache-mod-musicindex_0.99.5_powerpc.deb
 4cebcc618722167d97cd195bceb2d3ee 26764 web optional libapache2-mod-musicindex_0.99.5_powerpc.deb

-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.2.4 (GNU/Linux)

iD8DBQFBTBXzHjLD2rfS8GMRAjhuAJ9jPkKp7aaLeh76jBfFMF676gs7PQCfdWK5
tJbpNfyTmt5t/deSQZkOso8=
=r8UL
-----END PGP SIGNATURE-----

Message-ID: <email address hidden>
Date: Mon, 20 Sep 2004 10:44:32 -0700
From: "Andrew D. Clark" <email address hidden>
To: <email address hidden>
Subject: Re: Bug#271912 acknowledged by developer (Bug#271912: fixed in
 libapache-mod-musicindex 0.99.5)

All packages don't appear to be available yet:

The following packages have unmet dependencies:
  mod-musicindex-common: Depends: libapache-mod-musicindex (= 0.99.5)
but it is not going to be installed or
                                  libapache2-mod-musicindex (= 0.99.5)
but it is not going to be installed

Were only powerpc debs updated?

--On Saturday, September 18, 2004 04:33:07 AM -0700 Debian Bug Tracking
System <email address hidden> wrote:

> <email address hidden>

--
Andrew Clark
Campus Network Programmer
Office of Information Technology
University of California, Santa Barbara
<email address hidden> (805) 893-5311

Message-Id: <email address hidden>
Date: Mon, 20 Sep 2004 20:09:45 +0200
From: Thibaut VARENE <email address hidden>
To: "Andrew D. Clark" <email address hidden>
Cc: <email address hidden>
Subject: Re: Bug#271912: acknowledged by developer (Bug#271912: fixed in
 libapache-mod-musicindex 0.99.5)

On Mon, 20 Sep 2004 10:44:32 -0700
"Andrew D. Clark" <email address hidden> wrote:

> All packages don't appear to be available yet:
>
> The following packages have unmet dependencies:
> mod-musicindex-common: Depends: libapache-mod-musicindex (= 0.99.5)
> but it is not going to be installed or
> libapache2-mod-musicindex (= 0.99.5)
>
> but it is not going to be installed
>
> Were only powerpc debs updated?
>
Yes.

The buildds are currently messed up because of broken apache2
dependency. You have to wait a bit for the package to get built again.

Thx

Besides, this kind of message shouldn't actually surprise you if you're
using unstable...

Thibaut VARENE
The PA/Linux ESIEE Team
http://www.pateam.org/

Message-ID: <email address hidden>
Date: Mon, 20 Sep 2004 11:57:25 -0700
From: "Andrew D. Clark" <email address hidden>
To: Thibaut VARENE <email address hidden>
cc: <email address hidden>
Subject: Re: Bug#271912: acknowledged by developer (Bug#271912: fixed
 in libapache-mod-musicindex 0.99.5)

--On Monday, September 20, 2004 08:09:45 PM +0200 Thibaut VARENE
<email address hidden> wrote:

> On Mon, 20 Sep 2004 10:44:32 -0700
> "Andrew D. Clark" <email address hidden> wrote:
>
>> All packages don't appear to be available yet:
>>
>> The following packages have unmet dependencies:
>> mod-musicindex-common: Depends: libapache-mod-musicindex (=
>> 0.99.5) but it is not going to be installed or
>> libapache2-mod-musicindex (=
>> 0.99.5)
>>
>> but it is not going to be installed
>>
>> Were only powerpc debs updated?
>>
> Yes.
>
> The buildds are currently messed up because of broken apache2
> dependency. You have to wait a bit for the package to get built again.
>
> Thx
>
> Besides, this kind of message shouldn't actually surprise you if
> you're using unstable...
>
>

Not surprised! I was just checking if the i386 builds got lost in the
shuffle.

--
Andrew Clark
Campus Network Programmer
Office of Information Technology
University of California, Santa Barbara
<email address hidden> (805) 893-5311