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，设置有效期，这样比较合理。