Bug#897377: qa.debian.org: Please add '_' in npmjs_project_char_re (fakeupstream.cgi)

[Xavier Guimard <x.guimard@free.fr>]

Package: qa.debian.org
Severity: normal
Tags: patch

Hi all,

npmjs.com authorizes project to have a '_' in their name. For example
require-optional and require_optional are 2 different projects. Sadly a
lot af npmjs projects don't have valid git tags in their GitHub
repository. That's why fakeupstream.cgi is very useful.

So could you authorize '_' for npmjs.org?

Best regards,
Xavier

--- 8< ---
--- fakeupstream.cgi.orig       2018-05-01 19:20:32.099036786 +0200
+++ fakeupstream.cgi    2018-05-01 19:13:07.621394749 +0200
@@ -47,7 +47,7 @@

 our $project_char_re = '[a-zA-Z0-9\-]';
 our $pypi_project_char_re = '[a-zA-Z0-9\-\.]';
-our $npmjs_project_char_re = '[a-zA-Z0-9\-\.]';
+our $npmjs_project_char_re = '[a-zA-Z0-9\-\._]';
 our $cratesio_project_char_re = '[a-zA-Z0-9_.-]';
 our $github_project_char_re = '[-_.a-zA-Z0-9]';
 our $hg_repository_re = '[a-zA-Z0-9\.\-:/]+';
--- >8 ---

-- System Information:
Debian Release: buster/sid
  APT prefers testing
  APT policy: (900, 'testing'), (500, 'unstable')
Architecture: amd64 (x86_64)
Foreign Architectures: i386

Kernel: Linux 4.15.0-2-amd64 (SMP w/4 CPU cores)
Locale: LANG=fr_FR.UTF-8, LC_CTYPE=fr_FR.UTF-8 (charmap=UTF-8), LANGUAGE= (charmap=UTF-8)
Shell: /bin/sh linked to /bin/dash
Init: systemd (via /run/systemd/system)
LSM: AppArmor: enabled