CKEditor/CKFinder升级心得

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

一、CKFinder的若干问题

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

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

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

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

在下面加一行代码:

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

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

3.上传安全问题

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

public override bool CheckAuthentication()
{
   return false;
}

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

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

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

3.2 文件扩展名校验

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

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代码的留言(或产品编码，个人简介编辑等)，写上这样一行代码

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

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

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

定位到Settings\ResourceType.cs，找到

public string[] AllowedExtensions;
public string[] DeniedExtensions;

再增加二个数组

public string[] AllowedMIMETypes;
public string[] DeniedMIMETypes;

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

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

然后再增加一个方法:

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，找到:

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

然后加上:

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

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

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里加上限制就行了

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()方法，改成下面这样:

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来动态处理

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改成下面这样

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

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

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的设置

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源代码