Closed
Bug 6744
Opened 25 years ago
Closed 25 years ago
profile created has %20 in string rather than a space
Categories
(Core Graveyard :: Profile: BackEnd, defect, P2)
Tracking
(Not tracked)
VERIFIED
FIXED
M11
People
(Reporter: Brade, Assigned: sspitzer)
References
Details
I ran AppRunnerDebug today and was prompted (first time) to create a profile.
For the profile name I entered a string with a space in it: "Mozilla Brade"
The actual folder created was named "Mozilla%20Brade"
Problems:
* Difficult to find from the OS
* Makes name even longer (file system on Mac is already limited to 31
characters)
Currently all special characters are being escaped.
So only alphabets and numericals can be used. We
will unescape the special characters in M7.
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
Target Milestone: M7 → M9
Reporter | ||
Updated•25 years ago
|
Status: RESOLVED → REOPENED
Reporter | ||
Comment 3•25 years ago
|
||
Reopening.
This is not fixed with my build from this morning.
Reporter | ||
Updated•25 years ago
|
Resolution: FIXED → ---
Updated•25 years ago
|
Priority: P3 → P2
Comment 5•25 years ago
|
||
This bug cause loading mail folders to fail. Bumping priority to P2, cc:ing
putterman and ducarroz.
Comment 6•25 years ago
|
||
Adding lchiang to the cc list. We need this fixed before we can have widespread
mac use, and we should probably put in the dialog some place that you can't
create a profile name with a space in it.
The target milestone shows M9. I'll add it to the M8 release notes tracking
bug for verah.
Comment 10•25 years ago
|
||
Bug 12174 is related in that profiles created before the escaping change can't
be accessed after the change.
Status: REOPENED → RESOLVED
Closed: 25 years ago → 25 years ago
Resolution: --- → WORKSFORME
Updated•25 years ago
|
Status: RESOLVED → REOPENED
Comment 11•25 years ago
|
||
I don't see how this is working, creating a folder with a space in the name
creates a folder with %20 used for space. This does not show in Profile Manager,
but does show in Users50/grace%20bush
(using build 1999092708)
Comment 12•25 years ago
|
||
Seth, are you touching this area too? Looks like we're probably creating the
directory with the encoded name rather than the plain text name. Should be a
simple fix.
Assignee | ||
Updated•25 years ago
|
Assignee: gayatrib → sspitzer
Status: REOPENED → NEW
Assignee | ||
Comment 13•25 years ago
|
||
re-assign to me.
Comment 14•25 years ago
|
||
Clearing WORKSFORME resolution due to reopen of this bug.
Assignee | ||
Updated•25 years ago
|
Target Milestone: M10 → M11
Assignee | ||
Comment 15•25 years ago
|
||
moving to m11.
Assignee | ||
Updated•25 years ago
|
Status: NEW → ASSIGNED
Assignee | ||
Comment 16•25 years ago
|
||
accepting. marking all milestone 11.
Assignee | ||
Updated•25 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 25 years ago → 25 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 17•25 years ago
|
||
this should be fixed now.
If you created a profile "Foo Bar", it would be "Users50:Foo Bar"
there are a bunch of potential bugs lurking in this code.
1) what if the profile name is too long for the native file system? (we need
to hash the name of the directory on disk, if necessary)
2) what if the profile matches a directory that already exists?
(we need to add a case insensitive check for existance. FOOBAR and foobar are
the same directory on windows.)
3) what about special characters? (can the user ".." or "/" themselves into
trouble?)
marking this fixed, and I'll log the other issues as bugs and fix them.
Updated•25 years ago
|
Status: RESOLVED → VERIFIED
Comment 18•25 years ago
|
||
this is fixed in 1999100713 on the mac
Comment 19•25 years ago
|
||
Moving all Profile Manager bugs to new Profile Manager Backend component.
Profile Manager component to be deleted.
Updated•9 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•