https/http fetch of a non-existent repository invokes authentication
Bug #1716745 reported by
Nish Aravamudan
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
git-ubuntu |
Fix Released
|
High
|
Nish Aravamudan |
Bug Description
We recently switched from git:// to https://. This is good for security, but https:// behaves differently for non-existing repositories, because the remote server can't detect if the repository is private or non-existent.
Related branches
~nacc/git-ubuntu:gu-review
- Server Team CI bot: Approve (continuous-integration)
- Andreas Hasenack: Approve
- Robie Basak: Pending requested
-
Diff: 912 lines (+282/-77)14 files modifiedgitubuntu/__main__.py (+3/-1)
gitubuntu/build.py (+14/-7)
gitubuntu/buildsource.py (+1/-1)
gitubuntu/clone.py (+8/-5)
gitubuntu/importer.py (+34/-21)
gitubuntu/importlocal.py (+8/-6)
gitubuntu/importppa.py (+5/-3)
gitubuntu/lint.py (+7/-5)
gitubuntu/merge.py (+15/-11)
gitubuntu/queue.py (+4/-2)
gitubuntu/remote.py (+9/-3)
gitubuntu/review.py (+156/-0)
gitubuntu/submit.py (+12/-8)
gitubuntu/tag.py (+6/-4)
~nacc/git-ubuntu:bugfixes-after-refactor
- Andreas Hasenack: Approve
- Server Team CI bot: Needs Fixing (continuous-integration)
-
Diff: 239 lines (+85/-51)2 files modifiedgitubuntu/git_repository.py (+59/-36)
gitubuntu/versioning.py (+26/-15)
~nacc/git-ubuntu:gu-review
- Andreas Hasenack: Needs Fixing
- Server Team CI bot: Approve (continuous-integration)
- Robie Basak: Pending requested
-
Diff: 873 lines (+230/-348)9 files modifiedgitubuntu/__main__.py (+2/-0)
gitubuntu/clone.py (+0/-20)
gitubuntu/git_repository.py (+62/-36)
gitubuntu/importer.py (+0/-87)
gitubuntu/merge.py (+0/-149)
gitubuntu/remote.py (+0/-18)
gitubuntu/review.py (+140/-0)
gitubuntu/submit.py (+0/-23)
gitubuntu/versioning.py (+26/-15)
Changed in usd-importer: | |
status: | New → In Progress |
importance: | Undecided → High |
assignee: | nobody → Nish Aravamudan (nacc) |
milestone: | none → 1.0 |
Changed in usd-importer: | |
status: | In Progress → Fix Released |
To post a comment you must log in.