Skip to content

get_developer_information_url: Please query user info by login, not email

Originally reported at !77 (closed) .

DDPO page can query with either email=email_address or login=email_address. In the latter case, the query can also show other packages maintained by the same maintainer but under a different email address. This change would use the login= query and show the complete information about packages under one maintainer name.

Since the original Merge Request is stalled, I am converting that with an issue so that it better reflects that it is a feature request, not a bug report.

The full patch is pasted as below.

commit 1775b48642ecbbc63a26332a234e6174f0ee71f3
Author: Boyuan Yang <byang@debian.org>
Date:   Sun Mar 10 13:33:16 2024 -0400

    rules: get_developer_info_url: query by login, not email
    
    DDPO page can query with either email=email_address or
    login=email_address. In the latter case, the query
    can also show other packages maintained by the same
    maintainer but under a different email address. This
    change would use the login= query and show the complete
    information about packages under one maintainer name.

diff --git a/distro_tracker/vendor/debian/rules.py b/distro_tracker/vendor/debian/rules.py
index 0be50cc9..c567b6ae 100644
--- a/distro_tracker/vendor/debian/rules.py
+++ b/distro_tracker/vendor/debian/rules.py
@@ -277,7 +277,7 @@ def get_developer_information_url(developer_email):
     """
     Return a URL to extra information about a developer, by email address.
     """
-    URL_TEMPLATE = 'https://qa.debian.org/developer.php?email={email}'
+    URL_TEMPLATE = 'https://qa.debian.org/developer.php?login={email}'
     return URL_TEMPLATE.format(email=quote_plus(developer_email))
 
 
diff --git a/distro_tracker/vendor/debian/tests.py b/distro_tracker/vendor/debian/tests.py
index 39cb73de..fcf91f1e 100644
--- a/distro_tracker/vendor/debian/tests.py
+++ b/distro_tracker/vendor/debian/tests.py
@@ -538,12 +538,12 @@ class GetDeveloperInformationSiteUrlTest(SimpleTestCase):
         developer_email = 'debian-dpkg@lists.debian.org'
         self.assertEqual(
             'https://qa.debian.org/developer.php'
-            '?email=debian-dpkg%40lists.debian.org',
+            '?login=debian-dpkg%40lists.debian.org',
             get_developer_information_url(developer_email))
 
         developer_email = 'email@domain.com'
         self.assertEqual(
-            'https://qa.debian.org/developer.php?email=email%40domain.com',
+            'https://qa.debian.org/developer.php?login=email%40domain.com',
             get_developer_information_url(developer_email)
         )