[2.1 API PackageRepositories] API doesn't provide arches validation at all allowing any name to be saved
Bug #1613704 reported by
Andres Rodriguez
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
MAAS |
Fix Released
|
Critical
|
Jeffrey C Jones |
Bug Description
I did the following:
maas admin package-repository update 3 arches=amd64,i386
And it saved as:
"arches": [
],
Related branches
lp:~trapnine/maas/fix-1613704-repo-api-validation
- Jeffrey C Jones (community): Approve
- Blake Rouse (community): Approve
-
Diff: 245 lines (+145/-14)5 files modifiedsrc/maasserver/api/tests/test_packagerepositories.py (+4/-1)
src/maasserver/forms_packagerepository.py (+34/-1)
src/maasserver/models/packagerepository.py (+2/-1)
src/maasserver/testing/factory.py (+3/-1)
src/maasserver/tests/test_forms_packagerepository.py (+102/-10)
Changed in maas: | |
milestone: | none → 2.1.0 |
assignee: | nobody → Jeffrey C Jones (trapnine) |
Changed in maas: | |
milestone: | 2.0.1 → none |
milestone: | none → 2.1.0 |
summary: |
- [2.1 API PackageRepositories] API is incorrectly saving arches as a - single string instead of separating them into a list + [2.1 API PackageRepositories] API doesn't provide arches validation at + all allowing any name to be saved |
Changed in maas: | |
importance: | Undecided → Critical |
status: | New → Triaged |
Changed in maas: | |
status: | Triaged → In Progress |
Changed in maas: | |
status: | In Progress → Fix Committed |
Changed in maas: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.