华子的代码空间

逆水行舟,不进则退。 关注系统编程、网络编程、并发、分布式。

tornado中计算etag的一个BUG

在tornado中是根据send buffer中的内容计算etag的,而这样对于一些只判断逻辑的页面会有问题。

计算etag的代码如下:

 1 def compute_etag(self):
 2     """Computes the etag header to be used for this request.
 3 
 4     May be overridden to provide custom etag implementations,
 5     or may return None to disable tornado's default etag support.
 6     """
 7     hasher = hashlib.sha1()
 8     for part in self._write_buffer:
 9         hasher.update(part)
10     return '"%s"' % hasher.hexdigest()

可以很清楚的看到,是根据self._writer_buffer中的内容计算etag,然后返回。

但是如果遇到下面的代码会出现问题:

1 def post(self):
2     username = self.get_argument("username")
3     if not redis.UserManager.is_user_exists(username):
4         self.set_secure_cookie("user", username)
5         self.redirect("/room", status=301)
6         return
7     else:
8         self.render("login.html", user_exists=True)

这段代码中,返回的内容永远不会有变化。 所以客户端判断返回为304,导致出现问题。 

解决方法是禁用etag。解决方法很简单,只要重载compute_etag方法,并返回None即可。

并且建议,对于所有动态页面,不用计算etag的方法,这样可以让客户端永远获取最新的内容。

如果真的要客户端缓存页面,可以用Cache-Control,设置有效期,这样比较合理。

 

 

posted on 2013-03-06 15:16  华子的代码空间  阅读(717)  评论(0编辑  收藏  举报

导航