转： CKEditor/CKFinder升级心得

CKEditor/CKFinder升级心得

这几天把一个旧项目中的fckeditor升级为ckeditor 3.2 + ckfinder 1.4.3 组合，下面是一些升级心得:

一、CKFinder的若干问题

1.单独使用
ckfinder从原fckeditor分离出来以后可以单独使用，通常我习惯于在工具栏中添加ckfinder.dll，这样以后要使用ckfinder直接从工具箱拖出来即可.
 
拖到页面中后，会形成这样一个控件实例：

?

1	<CKFinder:FileBrowser ID="FileBrowser1" runat="server"></CKFinder:FileBrowser>

2.上传文件自动重命名
修改ckfinder的源文件，找到Connector\CommandHandlers\FileUploadCommandHandler.cs这个文件，定位到:

?

1 2	string sExtension = System.IO.Path.GetExtension( oFile.FileName ); sExtension = sExtension.TrimStart( '.' );

在下面加一行代码:

?

1	sFileName = DateTime.Now.ToString("yyyyMMddHHmmssfff") + "." + sExtension;

即强制把文件名改为时间格式字符串.

3.上传安全问题

3.1 跟fckeditor类似，默认情况下ckfinder是不允许上传的，找到config.ascx这个文件，定位到

?

1 2 3 4

public override bool CheckAuthentication() {    return false; }

 在这里加入自己需要的判断逻辑，千万不要直接改成return true;这样相当于免费把自己的服务器变成一个网络硬盘+肉鸡，任何人都可以直接上传任何文件(包括木马)，起码也得类似下面这样:

?

1 2 3 4

public override bool CheckAuthentication() {    return HttpContext.Current.User.Identity.IsAuthenticated; }

如果您是用membership/role来认证的，上面代码要求用户登录后才能使用ckfinder的上传功能.

3.2 文件扩展名校验

默认情况下，ckfinder几乎能上传任何文件，所以设置允许上传的文件扩展名是必需的，ckfinder采用了黑白名单的做法，即同时可以设置"允许上传的扩展名"及"禁止上传的扩展名"，config.ascx中可参考下面这样设置:

?

1 2 3 4 5 6 7

ResourceType type;   type = ResourceType.Add("Zip"); ... type.AllowedExtensions = new string[] { "zip" }; type.DeniedExtensions = new string[] {"asp","aspx","jsp","php","ashx","js","html","htm" }; ...

这一段设置相当于只允许.zip文件上传，同时禁止.asp,.aspx...之类的服务端文件上传

3.3 MIME类型/ContentType校验

光有扩展名校验是远远不够的，比如在asp时代就有一种经典的攻击方式:

a.先把asp木马文件扩展名改成.jpeg之类(这样就能绕过扩展名检验)
b.然后利用其它发包工具(或直接用ckfinder的上传功能)，上传"伪jpeg"文件
c.如果网站还支持html代码的留言(或产品编码，个人简介编辑等)，写上这样一行代码

?

1	<!--inlude file = "xxx.jpeg"-->

这里xxx.jpeg即上传后的"伪jpeg"木马，如果服务端允许包含文件的话，浏览包含这行代码的页面，木马就能运行了！

为了防止这类攻击，必须要在服务端做MIME/ContentType校验，因为文件的扩展名不管改成什么，其内在的MIME/ContentType是不会变的，修改方法：

定位到Settings\ResourceType.cs，找到

?

1 2	public string[] AllowedExtensions; public string[] DeniedExtensions;

再增加二个数组

?

1 2	public string[] AllowedMIMETypes; public string[] DeniedMIMETypes;

相应的构造函数也加初始化代码:

?

1 2	AllowedMIMETypes = new string[0]; DeniedMIMETypes = new string[0];

然后再增加一个方法:

?

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21

public bool CheckMIMEType(string mimeType) {     mimeType = mimeType.Trim().ToLower();       if (DeniedMIMETypes.Length > 0)     {     if (Array.IndexOf(this.DeniedMIMETypes, mimeType) >= 0)     {         return false;     }     }       if (AllowedMIMETypes.Length > 0)     {     return (Array.IndexOf(this.AllowedMIMETypes, mimeType) >= 0);     }     else     {     return true;     } }

然后定位到Connector\CommandHandlers\FileUploadCommandHandler.cs，找到:

?

1 2 3 4

if (!this.CurrentFolder.ResourceTypeInfo.CheckExtension(sExtension)) {     ConnectorException.Throw(Errors.InvalidExtension); }

然后加上:

?

1 2 3 4 5

string sFileMIME = oFile.ContentType; if (!this.CurrentFolder.ResourceTypeInfo.CheckMIMEType(sFileMIME))//检测上传文件的MIME类型 {     ConnectorException.Throw(Errors.InvalidMIMEType); }

最后再修改config.ascx，加上MIME类型的黑白名单:

?

1 2 3 4 5 6 7 8

ResourceType type;   type = ResourceType.Add("Zip"); ... type.AllowedExtensions = new string[] { "zip" }; type.DeniedExtensions = new string[] {"asp","aspx","jsp","php","ashx","js","html","htm" }; type.AllowedMIMETypes = new string[] { "application/x-zip-compressed" }; type.DeniedMIMETypes = new string[] {"text/plain" };

这样就相对就安全一些了（当然服务器端还可以进一步做安全处理，不过这个话题再展开就变成"服务器安全设置"专题了，不在本文的讨论范围，暂不深入）

4.上传文件大小限制

默认情况下ResourceType的构造函数里，MaxSize=0即不对上传文件大小做限制，所以只要在config.ascx里加上限制就行了

?

1 2 3

type = ResourceType.Add("Zip"); ... type.MaxSize = 0;

即把这里的MaxSize改成想要的值即可(以字节为单位计算)，注意：ResourceType 虽然有MaxSize成员，但其实上传代码中并未对上传文件大小做判断，而是在上传完成后生成缩略图时，才做了一次判断，如果需要在上传文件SaveAs 以前就做判断处理，自行加一条if语句，比较oFile.ContentLength与MaxSize即可

5.上传后缩略图无法正常显示

这是ckFinder在windows系统中的一个小bug，定位到Settings\Thumbnails.cs,找到public string GetTargetDirectory()方法，改成下面这样:

?

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15

if (Dir.Length == 0 || Dir.Substring(0,1)!="/") //如果Dir为空，或者只是相对路径 {     return HttpContext.Current.Server.MapPath(Url); } else {     if (Dir.IndexOf(":\\") == -1)//如果不是物理路径     {         return HttpContext.Current.Server.MapPath(Dir);     }     else     {         return Dir;     } }

6.动态指定上传路径

默认情况下无法用cs代码修改config.ascx中的BaseUrl设置，因为其后端代码ConfigFile中并没有提供修改BaseUrl的方法，这里我借用了fckeditor以前的用法：利用session来动态处理

?

1 2 3 4 5 6 7 8 9 10 11 12 13

public string DynamicBaseUrl {                get     {     object _baseUrl = HttpContext.Current.Session["CKFinder:DynamicBaseUrl"];     if (_baseUrl == null || string.IsNullOrEmpty(_baseUrl.ToString()))     {         _baseUrl = "/ckfinder/userfiles/";     }     this.BaseUrl = _baseUrl.ToString();     return this.BaseUrl;     } }

如上，在Settings\ConfigFile.cs中增加一个属性，让其从session中取值，然后再把config.ascx中的BaseUrl改成下面这样

?

1 2	//BaseUrl = "/ckfinder/userfiles/"; BaseUrl = DynamicBaseUrl;

最后在嵌入ckFinder的页面中类似这样处理：

?

1 2 3 4

protected void Page_Load(object sender, EventArgs e) {     Session["CKFinder:DynamicBaseUrl"] = "/upload/"; }

7.CKfinder免费版本如何去掉“那啥”的提示

打开core\js\ckfinder_ie.js，找到 {en.call(window,qo);}，改成{/*en.call(window,qo);*/}即可

二、与CKeditor的整合

1.CKeditor的设置

?

1 2 3 4 5 6 7 8 9 10

window.onload = function () {     CKEDITOR.replace("editor1", {     filebrowserBrowseUrl: '/ckfinder/ckfinder.html', //启用浏览功能     filebrowserImageBrowseUrl: '/ckfinder/ckfinder.html?Type=Image',     filebrowserFlashBrowseUrl: '/ckfinder/ckfinder.html?Type=Flash',     filebrowserUploadUrl: '/ckfinder/core/connector/aspx/connector.aspx?command=QuickUpload&type=Zip',     filebrowserImageUploadUrl: '/ckfinder/core/connector/aspx/connector.aspx?command=QuickUpload&type=Image',     filebrowserFlashUploadUrl: '/ckfinder/core/connector/aspx/connector.aspx?command=QuickUpload&type=Flash'     }); }

这样就可以了，需要说明的"ckfinder.html?Type=Image"上的Type=XXX，即对应CKFinder中 Config.ascx的ResourceType设置，而且ResourceType的名称不能用中文名，否则在快速上传时无法上传到服务端。(很多地 方是在html中以js方式接收参数的，改成中文后会导致乱码，从而无法正确定位目录，熟悉js的朋友如果想让其支持中文Type名，技术上讲应该是可以 修改实现的)

2.与Asp.Net默认安全性的冲突处理

可参见上一篇博文，不再重复

最后：CKFinder需要Session/ViewState，所以如果您的Asp.Net项目中禁用了Session或ViewState，可能会无法正常运行，解决办法要么启用Session/ViewState，要么自行修改CKFinder源代码