Opened at 2008-04-03T10:49:14Z
Closed at 2011-01-03T07:32:00Z
#376 closed defect (wontfix)
locale-bug in the SMB (WinFUSE) frontend
| Reported by: | MartinH | Owned by: | |
|---|---|---|---|
| Priority: | major | Milestone: | eventually |
| Component: | code-frontend | Version: | 1.0.0 |
| Keywords: | windows smb | Cc: | |
| Launchpad Bug: |
Description
There seems to be a locale-bug in the WinFUSE interface (in TahoeFS. dll, in the method LinkFiles? in class TahoeInterface?).
It generates an invalid JSON message for the request "set_children" when the current culture uses a comma as the decimal separator character, as is the case in Sweden. This causes the request to fail in webish.py, line 996 (children = simplejson.loads(body)) with a ValueError?('Expecting property name: line 1 column 200 (char 200)')
This is becuase the generated JSON for ctime/mtime ends up with an unexpected comma as the decimal separator:
{ "cryptopp552.zip": [ "filenode", { "ro_uri": "URI:CHK:
hsll4q3uroho5vp2jvywj7ujpe:
egi3wuth5n4bacn3hllcujhfdy4uzb5ctj3tvfwm56ho7qqk4haq:3:10:992073",
"size": 992073, "metadata": { "ctime": 1207150752,03319, "mtime":
1207150752,03319 } } ] }
I could not find the source for TahoeFS.dll in the repository, but using Reflector it seems that the ToString?() call converting the double "TotalSeconds?" into a string should explicitly specify CultureInfo?.InvariantCulture? instead of implicitly using the current culture, as in span.TotalSeconds?.ToString?(CultureInfo?.InvariantCulture?).
Change History (8)
comment:1 Changed at 2009-03-17T02:10:12Z by zooko
- Component changed from code-frontend to code-frontend-w32
comment:2 Changed at 2009-06-21T19:33:46Z by zooko
- Keywords fuse added
comment:3 Changed at 2009-06-21T19:34:23Z by zooko
- Keywords fuse removed
comment:4 Changed at 2010-01-09T20:49:04Z by davidsarah
- Keywords windows winfuse added; WinFUSE removed
comment:5 Changed at 2010-02-11T00:36:17Z by davidsarah
- Keywords smb added; winfuse removed
- Summary changed from locale-bug in the WinFUSE interface to locale-bug in the SMB (WinFUSE) frontend
comment:6 Changed at 2010-03-21T16:49:45Z by davidsarah
- Component changed from code-frontend-w32 to code-frontend
comment:7 Changed at 2011-01-03T05:22:15Z by davidsarah
If we have given up on getting the NeoGeo? SMB library open-sourced, then we should close this ticket.
comment:8 Changed at 2011-01-03T07:32:00Z by davidsarah
- Resolution set to wontfix
- Status changed from new to closed
simplifying component categories (use 'smb' keyword for WinFUSE)