unserialized Testament.as_text() doesn't handle unicode revprops
Bug #54723 reported by
John A Meinel
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Bazaar |
Fix Released
|
Medium
|
John A Meinel | ||
Bug Description
I haven't spent any time to track down the specific bug, but the attached patch updates the test suite, which failes with a UnicodeError
I *think* the bug is that revprops are double serialized. (they are a stanza within a stanza).
So every time a stanza is parsed, it is decoded, even if it has already been decoded.
So it probably is a pure Testament bug, but the attached patch exposes it.
Related branches
Revision history for this message
| John A Meinel (jameinel) wrote | #1 |
Revision history for this message
| John A Meinel (jameinel) wrote | #2 |
| Changed in bzr: | |
| assignee: | nobody → jameinel |
| importance: | Untriaged → Medium |
| status: | Unconfirmed → In Progress |
Revision history for this message
| John A Meinel (jameinel) wrote | #3 |
| Changed in bzr: | |
| status: | In Progress → Fix Committed |
| Changed in bzr: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
I came across this while fixing some other revprop code paths