Activity log for bug #1821165

Date Who What changed Old value New value Message
2019-03-21 10:16:05 Robin Åstedt bug added bug
2019-03-21 10:16:05 Robin Åstedt attachment added Proposed bug fix https://bugs.launchpad.net/bugs/1821165/+attachment/5248185/+files/fix.patch
2019-03-21 11:13:47 Robin Åstedt description In texttestlib/default/batch/jenkinschanges.py method parsePlainChangeLog: Here it parses changelog.xml (When it is in plain text format) by opening it in utf-8 encoding, but then when converting from a unicode string to a byte string it specifies ascii encoding. Proposed fix: - self.addUnique(bugs, self.getBugs(line.encode("ascii"))) + self.addUnique(bugs, self.getBugs(line.encode("utf-8"))) This information is then used to generate an url so perhaps it would be good to strip the string of any non-url-safe characters. But, calling line.encode("ascii") for a file opened as utf-8 is definitely wrong. Please note that this blocks the generation of the reports in our Jenkins jobs, and I could only fix it via a live edit to the installed library, so I'd appreciate if this could be looked on soon. Thanks! In texttestlib/default/batch/jenkinschanges.py method parsePlainChangeLog: Here it parses changelog.xml (When it is in plain text format) by opening it in utf-8 encoding, but then when converting from a unicode string to a byte string it specifies ascii encoding. Proposed fix: - self.addUnique(bugs, self.getBugs(line.encode("ascii"))) + self.addUnique(bugs, self.getBugs(line.encode("utf-8"))) This information is then used to generate an url so perhaps it would be good to strip the string of any non-url-safe characters. But, calling line.encode("ascii") for a file opened as utf-8 is definitely wrong.
2019-03-21 21:40:14 Geoff Bache texttest: status New Fix Committed
2019-03-21 21:40:14 Geoff Bache texttest: assignee Geoff Bache (geoff.bache)
2019-03-25 12:14:00 Geoff Bache texttest: status Fix Committed Fix Released