Pillar name search raises IndexError if summary contains '%'
Bug #1921311 reported by
Jacob Duncan
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
High
|
Colin Watson | ||
Bug Description
OOPS-52836de3204e4e3f69e872ed6a3b32c1
Related branches
~cjwatson/launchpad:sqlvalues-text
- Thiago F. Pappacena (community): Approve
-
Diff: 723 lines (+180/-157)17 files modifiedlib/lp/bugs/model/structuralsubscription.py (+5/-5)
lib/lp/oci/tests/test_ocirecipe.py (+7/-2)
lib/lp/registry/model/distribution.py (+2/-4)
lib/lp/registry/model/distroseries.py (+1/-1)
lib/lp/registry/model/person.py (+2/-2)
lib/lp/registry/model/pillar.py (+42/-31)
lib/lp/registry/model/product.py (+9/-8)
lib/lp/registry/model/productseries.py (+4/-6)
lib/lp/registry/model/projectgroup.py (+17/-19)
lib/lp/registry/tests/test_pillar.py (+12/-0)
lib/lp/registry/vocabularies.py (+6/-7)
lib/lp/services/database/sqlbase.py (+7/-2)
lib/lp/services/database/stormexpr.py (+4/-3)
lib/lp/services/identity/model/emailaddress.py (+10/-4)
lib/lp/services/statistics/model/statistics.py (+0/-1)
lib/lp/soyuz/model/archive.py (+16/-17)
lib/lp/soyuz/scripts/gina/handlers.py (+36/-45)
Revision history for this message
| Cristian Gonzalez (cristiangsp) wrote | #1 |
| Changed in launchpad: | |
| status: | New → Confirmed |
Revision history for this message
| Colin Watson (cjwatson) wrote | #2 |
| summary: |
- OOPS-52836de3204e4e3f69e872ed6a3b32c1 + Pillar name search raises IndexError if summary contains '%' |
| Changed in launchpad: | |
| importance: | Undecided → High |
| status: | Confirmed → Triaged |
| tags: | added: lp-registry search |
| tags: | added: oops |
| Changed in launchpad: | |
| assignee: | nobody → Colin Watson (cjwatson) |
| status: | Triaged → In Progress |
| Changed in launchpad: | |
| status: | In Progress → Fix Committed |
| Changed in launchpad: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
It seems that the problem that caused the OOPS error was the length of the summary that you introduced when trying to create a new project.
On LP side we could do a better job handling that situation and provide guidance if we detect that the user is providing a too long summary.
Till that gets solved, you will be able to proceed if you substantially reduce the length of the summary of the project that you were trying to create.
Thanks for the report.