Opened 16 years ago

Closed 16 years ago

#1945 closed Bug (fixed)

Sanitize{File,Folder}Name does miss control characters

Reported by: Michael Osipov Owned by: Wiktor Walc
Priority: Must have (possibly next milestone) Milestone: FCKeditor 2.6
Component: File Browser Version: FCKeditor 2.6 Beta
Keywords: Cc:

Description

functions miss to replace http://en.wikipedia.org/wiki/ASCII#ASCII_control_characters I did in Java with a simple "|[\u0000-\u001F]|\u007F"

Just add to PHP and other connectors

Change History (7)

comment:1 Changed 16 years ago by Frederico Caldeira Knabben

Milestone: FCKeditor 2.6

comment:2 Changed 16 years ago by Wiktor Walc

Owner: set to Wiktor Walc

To create similar pattern in PHP: \x{001F} utf-8 mode is required, it is available @Windows since 4.2.3 so to be fully compatible with such users I think it's better to simply use cntrl set in regular expression: [:cntrl:].

comment:3 Changed 16 years ago by Michael Osipov

wwalc,

great idea! I adapted it for Java with "\p{Cntrl}" too. I missed to check the api Thx

comment:4 Changed 16 years ago by Wiktor Walc

Component: Server : PHPFile Browser

PHP fixed with [1648], Perl: [1649], Python: [1651], ColdFusion: [1652].

comment:5 Changed 16 years ago by Wiktor Walc

It seems that it is impossible to implement such check in Lasso due to language limitations, however I have added just a bascic check of folder and file names with [1655].

Someone could fix this in asp & asp.net, then we can close this ticket.

comment:6 Changed 16 years ago by Alfonso Martínez de Lizarrondo

Asp fixed with [1672]

comment:7 Changed 16 years ago by Frederico Caldeira Knabben

Resolution: fixed
Status: newclosed

ASP.NET fixed with [1701]. Changelog entry added with [1702].

Note: See TracTickets for help on using tickets.
© 2003 – 2022, CKSource sp. z o.o. sp.k. All rights reserved. | Terms of use | Privacy policy