Ticket #1945 (closed Bug: fixed)

Opened 2 years ago

Last modified 2 years ago

Sanitize{File,Folder}Name does miss control characters

Reported by: mosipov Owned by: wwalc
Priority: High 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

Changed 2 years ago by fredck

  • milestone set to FCKeditor 2.6

Changed 2 years ago by wwalc

  • owner set to wwalc

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:].

Changed 2 years ago by mosipov

wwalc,

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

Changed 2 years ago by wwalc

  • component changed from Server : PHP to File Browser

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

Changed 2 years ago by wwalc

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.

Changed 2 years ago by alfonsoml

Asp fixed with [1672]

Changed 2 years ago by fredck

  • status changed from new to closed
  • resolution set to fixed

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

Note: See TracTickets for help on using tickets.