China News

为研究 Firefox 扩展开发整的 Zeal.ChinaStock 从去年5月份开始提请审查，到今天将近一年的时间，终于得见天日不用再待在沙盒里面接收测试了。自贺一下。

历次被打回的理由如下：

1. Your add-on must have some reviews either on AMO or elsewhere on the web. See http://shawnwilsher.com/archives/108 for some ideas on how to do that.

第一次的提请审核时间最久，在我都已经快忘了这回事的时候 AMO 回了这么一封信。看来必须是在沙盒里面待到有一定的回复评论数量才行。

2. In order to prevent conflicts with other addons that may be installed by users, you need to wrap your ‘loose’ variables and functions within a JavaScript object. You can see an example of how to do this @ http://blogger.ziesemer.com/2007/10/respecting-javascript-global-namespace.html.

第二次告诉我代码里面有用到全局变量，容易跟其他扩展的代码发生冲突。看了看其它公布在AMO网站上的附件组件，有不少也是存在大量全局变量的，估计要么是提交的早审核没这么严格，要么是影响力大AMO给开后门了。只好用Firebug查看所有的全局变量，把它们都封装到自己的命名空间对象下面。 AMO 提供的 how-to 链接是被墙的，需要翻墙阅读。

3. You’re evaluating remote code, which may be a serious security risk. Please use https://developer.mozilla.org/En/Components.utils.evalInSandbox and resubmit.

原来代码里面使用了 eval 来执行远程抓取到的内容。去掉了对 eval 的使用之后再提交，没过多久就收到 Congratulations 邮件了。

Updated 2009-04-02 11:50 — 更新了一次版本，差不多等了一个多星期之后 Mozilla 完成版本review予以发布。有意思的是他们居然同时还在 Minefield 上进行了测试，并建议我 increase max compatible version from the developer control pane (no need to resubmit with an altered install.rdf) ã€‚这直接促使我下载了还在开发中的  latest nightly build è¿›è¡Œæµ‹è¯•ã€‚Mozilla 的测试人员确实是认真负责加勤奋，赞一个。

This entry was posted on Tuesday, September 29th, 2009 at 1:50 pm and is filed under Uncategorized. You can follow any responses to this entry through the RSS 2.0 feed. You can leave a response, or trackback from your own site.