Debian Bug report logs - #712261
uscan: fails with alioth downloads: failed: 406 Not Acceptable

Toggle useless messages

Message #5 received at submit@bugs.debian.org (full text, mbox, reply):

From: Ludovic Rousseau <rousseau@debian.org>

To: Debian Bug Tracking System <submit@bugs.debian.org>

Subject: uscan: fails with alioth downloads: failed: 406 Not Acceptable

Date: Fri, 14 Jun 2013 18:39:35 +0200

Package: devscripts
Version: 2.12.6
Severity: important

Hello,

$ cat debian/watch 
version=3
https://alioth.debian.org/frs/?group_id=30105 .*/ccid-(\d.*)\.(?:tgz|tar\.(?:gz|bz2|xz))

$ uscan --report
uscan warning: In watchfile debian/watch, reading webpage
  https://alioth.debian.org/frs/?group_id=30105 failed: 406 Not Acceptable

After a tcpdump/wireshark analysis I get the following HTTP exchange:
GET /frs/?group_id=30105 HTTP/1.1
TE: deflate,gzip;q=0.3
Connection: TE, close
Host: alioth.debian.org
User-Agent: Debian uscan 2.12.6
X-Uscan-Features: enhanced-matching

HTTP/1.1 406 Not Acceptable
Date: Wed, 12 Jun 2013 12:20:01 GMT
Server: Apache/2.2.22 (Debian)
X-Powered-By: PHP/5.4.4-14
Vary: Accept-Encoding
Content-Length: 0
Content-Type: text/html
Connection: close


I guess the recent upgrade of alioth broke uscan.

-- Package-specific info:

--- /etc/devscripts.conf ---

--- ~/.devscripts ---
Not present

-- System Information:
Debian Release: 7.0
  APT prefers stable
  APT policy: (990, 'stable'), (500, 'unstable')
Architecture: amd64 (x86_64)

Kernel: Linux 3.2.0-4-amd64 (SMP w/1 CPU core)
Locale: LANG=fr_FR.UTF-8, LC_CTYPE=fr_FR.UTF-8 (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash

Versions of packages devscripts depends on:
ii  dpkg-dev  1.16.10
ii  libc6     2.13-38
ii  perl      5.14.2-21
ii  python    2.7.3-4

Versions of packages devscripts recommends:
ii  at                        3.1.13-2
ii  dctrl-tools               2.22.2
ii  debian-keyring            2013.04.21
ii  dput                      0.9.6.3+nmu2
ii  equivs                    2.0.9
ii  fakeroot                  1.18.4-2
ii  gnupg                     1.4.12-7
ii  libcrypt-ssleay-perl      0.58-1
ii  libdistro-info-perl       0.10
ii  libjson-perl              2.53-1
ii  libparse-debcontrol-perl  2.005-3
ii  libsoap-lite-perl         0.714-1
ii  liburi-perl               1.60-1
ii  libwww-perl               6.04-1
ii  lintian                   2.5.10.4
ii  man-db                    2.6.2-1
ii  patch                     2.6.1-3
ii  patchutils                0.3.2-1.1
ii  python-debian             0.1.21
ii  python-magic              5.11-2
ii  sensible-utils            0.0.7
ii  strace                    4.5.20-2.3
ii  unzip                     6.0-8
ii  wdiff                     1.1.2-1
ii  wget                      1.13.4-3
ii  xz-utils                  5.1.1alpha+20120614-2

Versions of packages devscripts suggests:
ii  bsd-mailx [mailx]            8.1.2-0.20111106cvs-1
ii  build-essential              11.5
pn  cvs-buildpackage             <none>
pn  devscripts-el                <none>
pn  gnuplot                      <none>
pn  libauthen-sasl-perl          <none>
ii  libfile-desktopentry-perl    0.04-3
pn  libnet-smtp-ssl-perl         <none>
pn  libterm-size-perl            <none>
ii  libtimedate-perl             1.2000-1
ii  libyaml-syck-perl            1.20-1
ii  mutt                         1.5.21-6.2
ii  openssh-client [ssh-client]  1:6.0p1-4
ii  svn-buildpackage             0.8.5
ii  w3m                          0.5.3-8

-- no debconf information

Message #10 received at 712261@bugs.debian.org (full text, mbox, reply):

From: Dmitry Smirnov <onlyjob@debian.org>

To: 712261@bugs.debian.org

Subject: Re: uscan: fails with alioth downloads: failed: 406 Not Acceptable [patch]

Date: Mon, 8 Jul 2013 02:35:18 +1000

Control: found -1 devscripts/2.13.2
Control: tags -1 confirmed patch

Apparently Alioth want "Accept: */*" header which could be trivially
fixed by the following patch:

--- a/usr/bin/uscan
+++ b/tmp/uscan
@@ -691,8 +691,9 @@
 
     # Comma-separated list of features that sites being queried might
     # want to be aware of
     $headers->header('X-uscan-features' => 'enhanced-matching');
+    $headers->header('Accept' => '*/*');
     %dehs_tags = ('package' => $pkg);
 
     if ($watch_version == 1) {
        ($site, $dir, $filepattern, $lastversion, $action) = split ' ', $line, 5;


-- 
Best wishes,
 Dmitry Smirnov
 GPG key : 4096R/53968D1B

Message #19 received at 712261@bugs.debian.org (full text, mbox, reply):

From: Dmitry Smirnov <onlyjob@member.fsf.org>

To: 712261@bugs.debian.org

Subject: Re: uscan: fails with alioth downloads: failed: 406 Not Acceptable [patch-fixed]

Date: Mon, 8 Jul 2013 02:58:40 +1000

Sorry, to actually download something from Alioth (not merely get a
list of files) another hunk is needed as below:

--- a/usr/bin/uscan
+++ b/tmp/uscan
@@ -691,8 +691,9 @@
 
     # Comma-separated list of features that sites being queried might
     # want to be aware of
     $headers->header('X-uscan-features' => 'enhanced-matching');
+    $headers->header('Accept' => '*/*');
     %dehs_tags = ('package' => $pkg);
 
     if ($watch_version == 1) {
        ($site, $dir, $filepattern, $lastversion, $action) = split ' ', $line, 5;
@@ -1313,9 +1314,11 @@
        }
        # substitute HTML entities
        # Is anything else than "&" required?  I doubt it.
        print STDERR "$progname debug: requesting URL $upstream_url\n" if $debug;
-       $request = HTTP::Request->new('GET', $upstream_url);
+       my $headers = HTTP::Headers->new;
+       $headers->header('Accept' => '*/*');
+       $request = HTTP::Request->new('GET', $upstream_url, $headers);
        $response = $user_agent->request($request, "$destdir/$newfile_base");
        if (! $response->is_success) {
            if (defined $pkg_dir) {
                uscan_warn "$progname warning: In directory $pkg_dir, downloading\n  $upstream_url failed: " . $response->status_line . "\n";


-- 
Regards,
 Dmitry Smirnov
 GPG key : 4096R/53968D1B

---

Free speech is the bedrock of liberty and a free society. And yes, it
includes the right to blaspheme and offend.
        -- Ayaan Hirsi Ali, 2010

Message #24 received at 712261-submitter@bugs.debian.org (full text, mbox, reply):

From: James McCoy <jamessan@debian.org>

To: 712261-submitter@bugs.debian.org

Subject: Bug#712261 marked as pending

Date: Fri, 12 Jul 2013 23:50:15 +0000

tag 712261 pending
thanks

Hello,

Bug #712261 reported by you has been fixed in the Git repository. You can
see the changelog below, and you can check the diff of the fix at:

    http://git.debian.org/?p=collab-maint/devscripts.git;a=commitdiff;h=bdb59b1

---
commit bdb59b1c005d0d627bcdc4bdbff663f3449e05e7
Author: James McCoy <jamessan@debian.org>
Date:   Fri Jul 12 19:48:25 2013 -0400

    uscan: Add "Accept: */*" header to fix downloading from Alioth.
    
    Closes: #712261
    Signed-off-by: James McCoy <jamessan@debian.org>

diff --git a/debian/changelog b/debian/changelog
index e5cb2e4..601e9ef 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -16,6 +16,8 @@ devscripts (2.13.3) UNRELEASED; urgency=low
     #712227)
   * debcommit: Use "git status --porcelain" to check for a dirty working
     directory.  (Closes: #712166)
+  * uscan: Add "Accept: */*" header to fix downloading from Alioth.  Thanks to
+    Dmitry Smirnov for the patch.  (Closes: #712261)
 
   [ Chris Boot ]
   * bts: Fix no-mutt option. (Closes: #709999)

Message #29 received at 712261-close@bugs.debian.org (full text, mbox, reply):

From: James McCoy <jamessan@debian.org>

To: 712261-close@bugs.debian.org

Subject: Bug#712261: fixed in devscripts 2.13.3

Date: Fri, 16 Aug 2013 00:33:05 +0000

Source: devscripts
Source-Version: 2.13.3

We believe that the bug you reported is fixed in the latest version of
devscripts, which is due to be installed in the Debian FTP archive.

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 712261@bugs.debian.org,
and the maintainer will reopen the bug report if appropriate.

Debian distribution maintenance software
pp.
James McCoy <jamessan@debian.org> (supplier of updated devscripts 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 ftpmaster@ftp-master.debian.org)


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

Format: 1.8
Date: Thu, 15 Aug 2013 20:00:57 -0400
Source: devscripts
Binary: devscripts
Architecture: source amd64
Version: 2.13.3
Distribution: unstable
Urgency: low
Maintainer: Devscripts Devel Team <devscripts-devel@lists.alioth.debian.org>
Changed-By: James McCoy <jamessan@debian.org>
Description: 
 devscripts - scripts to make the life of a Debian Package maintainer easier
Closes: 600048 610712 658999 704236 709999 711467 712166 712227 712261 712701 714599 714835 715572
Changes: 
 devscripts (2.13.3) unstable; urgency=low
 .
   [ gregor herrmann ]
   * debcommit: Add the source package name to commit/tag messages.  (Closes:
     #704236)
 .
   [ Paul Wise ]
   * In chdist bash completion, don't print warnings if ~/.chdist is missing
   * Support 'new' in the rmadison URL list.
 .
   [ James McCoy ]
   * dpkg-buildpackage: Use "dpkg-checkbuilddeps -A" for arch-indep builds.
     Bump dpkg-dev Depends to 1.16.4 accordingly.
   * Suppress "given/when is experimental" warnings when using Perl 5.18.
   * mk-build-deps: Take Build-Conflicts(-Indep) into account.  (Closes:
     #712227)
   * debcommit: Use "git status --porcelain" to check for a dirty working
     directory.  (Closes: #712166)
   * uscan: Add "Accept: */*" header to fix downloading from Alioth.  Thanks to
     Dmitry Smirnov for the patch.  (Closes: #712261)
   * transition-check: Work around Pod::Simple change which causes Perl 5.18's
     pod2man to fail.  (Closes: #714599)
   * licensecheck:
     + Check haskell files.  Thanks to Jens Petersen for the patch.
     + Handle differing lengths of comment leaders when autodetecting comments.
       (Closes: #714835)
   * uscan/rc-alert: Fix roff errors in man pages.  Thanks to Eric S. Raymond
     for the patch.  (Closes: #712701)
   * dscverify: Use ~/.gnupg/trustedkeys.gpg, if available.  (Closes: #600048)
 .
   [ Chris Boot ]
   * bts: Fix no-mutt option. (Closes: #709999)
 .
   [ Daniel Kahn Gillmor ]
   * Enable OpenPGP signature verification (Closes: #610712)
 .
   [ Yaroslav Halchenko ]
   * licensecheck: Check .go (Go) files (Closes: #711467)
   * wnpp-check: Add bug URL to output and reorder columns to ease reading.
     (Closes: #715572)
 .
   [ Bernhard R. Link ]
   * uscan: properly cope with colons and minus signs in upstream versions.
     (Closes: #658999)
 .
   [ Phillip Hands ]
   * licensecheck:
     + allow plural: copyright notices
     + ignore "copyright holders"
     + alternative BSD-3 wording "authors" and "_any_ contributors"
     + ignore #defines containing (c)
     + fix "copyright ownership"
     + ignore "copyright statement"
     + ignore "copyright string"
     + ignore coyright (claim|is|in), and fix test
     + ignore "copyright to"
Checksums-Sha1: 
 516c5bb5496f1ec7e0aa143ac7310121a9802f2f 1483 devscripts_2.13.3.dsc
 f2bb5d52f08a0b03583187bf7fb1e9f868966369 573368 devscripts_2.13.3.tar.xz
 af989b8067394ee26fcfe09a60e4b96b725ab6e9 854416 devscripts_2.13.3_amd64.deb
Checksums-Sha256: 
 e334049b8edf531756ad7b408dcec5f51e6e6df5f4720c02ca7ac9a93371a2fc 1483 devscripts_2.13.3.dsc
 216a8a63e0a6e4d5f677ceb52e6330aa90851471fe337e87f3d053a2c1538556 573368 devscripts_2.13.3.tar.xz
 8757b74d7d5390c7cad72aa77f07c68f7d8e9a6c4f08608886ecdb77e84a7519 854416 devscripts_2.13.3_amd64.deb
Files: 
 d98029fd3966299e01baa2bbed63cffc 1483 devel optional devscripts_2.13.3.dsc
 1273a87daf9ff4d47f079687c2262a8c 573368 devel optional devscripts_2.13.3.tar.xz
 eb575ee6b000fd1874fa16638ff98190 854416 devel optional devscripts_2.13.3_amd64.deb

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

iEYEARECAAYFAlINbqwACgkQDb3UpmEybUDnRACaA4iqJu9xYzZiKrJh+da0xrzP
224AmQE70ns7dpvTyQ6eXzaypADSsFm0
=i+ZD
-----END PGP SIGNATURE-----

Send a report that this bug log contains spam.