Merge ~xypron/ppa-dev-tools:urldecode into ppa-dev-tools:main
Proposed by
Heinrich Schuchardt
Status: | Superseded | ||||
---|---|---|---|---|---|
Proposed branch: | ~xypron/ppa-dev-tools:urldecode | ||||
Merge into: | ppa-dev-tools:main | ||||
Diff against target: |
73 lines (+20/-10) 2 files modified
ppa/job.py (+7/-6) tests/test_job.py (+13/-4) |
||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Bryce Harrington | Needs Information | ||
Review via email: mp+458485@code.launchpad.net |
This proposal has been superseded by a proposal from 2024-01-12.
Commit message
Job.request_url: urlencode parameter strings
Triggering autopkgtests for a package with a version number like 8.2313.
To post a comment you must log in.
Hi Heinrich,
Big thanks for proposing a fix along with the bug. Yes, this looks like a very good way to solve the problem and I agree with your approach.
Can you also include a unit test case along with this fix? It should fail without your fix applied, and pass with it (usual TDD stuff).
Thanks again for implementing the fix!