Page MenuHomeSoftware Heritage

Make user_agent a parameter of GitHubSession
ClosedPublic

Authored by vlorentz on Apr 25 2022, 3:55 PM.

Details

Summary

So it can be set when used by other packages

Addresses D7630#199322

Diff Detail

Event Timeline

Build is green

Patch application report for D7646 (id=27669)

Could not rebase; Attempt merge onto 2fa9f0abd2...

Updating 2fa9f0a..2bbb6c8
Fast-forward
 swh/lister/github/lister.py            | 169 ++------------------------------
 swh/lister/github/tests/test_lister.py |  15 +--
 swh/lister/github/utils.py             | 170 +++++++++++++++++++++++++++++++++
 3 files changed, 188 insertions(+), 166 deletions(-)
 create mode 100644 swh/lister/github/utils.py
Changes applied before test
commit 2bbb6c81022a316f7a2ce191440f2610d3ce1299
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Mon Apr 25 15:53:41 2022 +0200

    Make user_agent a parameter of GitHubSession
    
    So it can be set when used by other packages

commit 2a993230e6547ad19ce90f11e09c1a187c6c60a3
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Mon Apr 25 15:44:39 2022 +0200

    Move GitHubSession from github/lister.py to github/utils.py
    
    So it can be reused by other packages without importing lister.py itself

commit f91caedbb2ed5eca2572b7b5ebe1413692496f60
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Thu Apr 21 20:34:50 2022 +0200

    github: Refactor rate-limiting out of the GitHubLister class
    
    This will allow the GitHub Metadata Fetcher to reuse the logic
    by importing the GitHubSession class.

commit d0924f39d04e7b7128c55a116280e2c6b6ec5a00
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Thu Apr 21 20:32:45 2022 +0200

    github: Remove dead code
    
    Authentication is handled directly in the session

See https://jenkins.softwareheritage.org/job/DLS/job/tests-on-diff/498/ for more details.

This revision is now accepted and ready to land.Apr 25 2022, 4:22 PM
olasd added a subscriber: olasd.

Cool, thanks.

Build is green

Patch application report for D7646 (id=27700)

Could not rebase; Attempt merge onto 334c54091e...

Updating 334c540..d715aaf
Fast-forward
 swh/lister/github/lister.py            | 161 ++-----------------------------
 swh/lister/github/tests/test_lister.py |  15 +--
 swh/lister/github/utils.py             | 170 +++++++++++++++++++++++++++++++++
 3 files changed, 188 insertions(+), 158 deletions(-)
 create mode 100644 swh/lister/github/utils.py
Changes applied before test
commit d715aaf903bb517873ec9716e0cc397da1fc64cd
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Mon Apr 25 15:53:41 2022 +0200

    Make user_agent a parameter of GitHubSession
    
    So it can be set when used by other packages

commit 2d04244cc950bfad254b7337449d2fe20e36f610
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Mon Apr 25 15:44:39 2022 +0200

    Move GitHubSession from github/lister.py to github/utils.py
    
    So it can be reused by other packages without importing lister.py itself

commit 9ee4a99f152d9d6cbe5a6a09895ce9384660be46
Author: Valentin Lorentz <vlorentz@softwareheritage.org>
Date:   Thu Apr 21 20:34:50 2022 +0200

    github: Refactor rate-limiting out of the GitHubLister class
    
    This will allow the GitHub Metadata Fetcher to reuse the logic
    by importing the GitHubSession class.

See https://jenkins.softwareheritage.org/job/DLS/job/tests-on-diff/504/ for more details.