I found that if you click "New Sub-Folder" and enter "\test\folder" with paths, UbuntuOne Should not allow this

Bug #391211 reported by Panayiotis Savva
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ubuntu One Servers
Fix Released
Medium
Joshua Blount

Bug Description

When Creating a "New Sub-Folder"
and you enter something like "\new\Folder", the web interface allows for this. But it really shoudn't

---

At this point the model code prevents such a folder from being created, but the web UI still needs to provide better feedback about what is allowed.

(See ubuntuone.storageprotocol.dircontent for the validation rules and a regexp which might be adaptable to JavaScript.)

Paul Sladen (sladen)
affects: ubuntuone-client → ubunet
tags: added: webstorage
Revision history for this message
Joshua Hoover (joshuahoover) wrote :

I confirmed this in the web ui. I tested the following using the "New Sub-Folder" button in the web ui:

"\new\Folder" - Synced to desktop client
"/new/Folder" - Sync daemon reported an error ('/test/folder' isn't a child of '/home/jhoover/Ubuntu One/My Files')

We should definitely not allow the creation of folders with forward slashes in the name. The user should be alerted that the name is invalid due to the forward slashes and left with the prompt to try to rename the folder.

Changed in ubunet:
assignee: nobody → Joshua Blount (jblount)
importance: Undecided → Critical
milestone: none → w11
status: New → Confirmed
tags: added: foundations+
Revision history for this message
Lucio Torre (lucio.torre) wrote :

more controller work

Changed in ubunet:
assignee: Joshua Blount (jblount) → Tim Cole (tcole)
Elliot Murphy (statik)
Changed in ubunet:
milestone: w11 → w13-karmic-alpha3
Revision history for this message
Tim Cole (tcole) wrote :

Hm. So why are we treating backslashes as path separators?

Changed in ubunet:
milestone: w13-karmic-alpha3 → w15
Revision history for this message
John Lenton (chipaca) wrote :

If you are working on this, please set it to "in progress". If you are not, why not? Is it not critical? If so, please adjust it. If it *is* critical, fix it!

Thanks,

Changed in ubunet:
status: Confirmed → Triaged
Changed in ubunet:
milestone: w15-karmic → w17-karmic-alpha4
Tim Cole (tcole)
Changed in ubunet:
importance: Critical → Medium
description: updated
Changed in ubunet:
assignee: Tim Cole (tcole) → Joshua Blount (jblount)
Changed in ubunet:
milestone: w17-karmic-alpha4 → w19-karmic-alpha5
Changed in ubuntuone-servers:
milestone: w19-karmic-alpha5 → w21-karmic-alpha6
Martin Albisetti (beuno)
Changed in ubuntuone-servers:
status: Triaged → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.