librarian apache log parser failing to parse bogus line in access log
Bug #676489 reported by
Diogo Matsubara
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
High
|
Gary Poster | ||
Bug Description
As see on OOPS-1781C2045 a ValueError: too many values to unpack was raised while parsing the access log
Related branches
lp://qastaging/~gary/launchpad/bug676489
- Edwin Grubbs (community): Approve (code)
- Benji York (community): Approve (code*)
-
Diff: 79 lines (+43/-7)2 files modifiedlib/lp/services/apachelogparser/base.py (+13/-7)
lib/lp/services/apachelogparser/tests/test_apachelogparser.py (+30/-0)
| tags: | added: librarian oops |
| affects: | launchpad-dev-moin-theme → launchpad-foundations |
| Changed in launchpad-foundations: | |
| status: | New → Triaged |
| importance: | Undecided → Low |
| importance: | Low → High |
Revision history for this message
| Gary Poster (gary) wrote | #1 |
| Changed in launchpad-foundations: | |
| assignee: | nobody → Gary Poster (gary) |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote Bug fixed by a commit | #2 |
| Changed in launchpad-foundations: | |
| milestone: | none → 10.12 |
| tags: | added: qa-needstesting |
| Changed in launchpad-foundations: | |
| status: | Triaged → Fix Committed |
| tags: |
added: qa-ok removed: qa-needstesting |
Revision history for this message
| Robert Collins (lifeless) wrote | #3 |
Revision history for this message
| Gary Poster (gary) wrote | #4 |
| Changed in launchpad-foundations: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
I investigated and found the original URL on devpad. The problem is that the path has a space in it, so our split gets four elements instead of three. I have a quick branch.