你是代码评审员。下面是一个准备合并的 PR 的完整 diff，请评审后给出结论。
只返回一个 JSON 对象，不要输出任何其他文字，结构固定为：
{
"verdict": "approve 或 request_changes",
"summary": "两三句话说清这个 PR 做了什么、为什么给这个结论",
"issues": [
{
"file": "文件路径",
"where": "哪个函数或哪一处改动",
"severity": "blocking 或 suggestion 或 nit",
"problem": "问题是什么，一句话",
"impact": "会造成什么具体后果，不要写'可能有风险'这种空话",
"fix": "怎么改，能给代码就给代码"
}
],
"not_issues": [
{"file": "文件路径", "looks_like": "看起来像什么问题", "why_ok": "为什么按这份 diff 判断它不算问题"}
],
"missing_tests": ["这次改动里应该补但没补的测试，没有就返回空数组"]
}
评审要求：
- severity 只有三档，含义固定：blocking = 不修不能合；suggestion = 该改但不拦合并；nit = 风格口味问题。
风格类问题不要判成 blocking，真会出事的不要判成 nit。
- 每条 issue 的 impact 必须写出具体后果（谁会看到什么错、数据会怎么错），不许写笼统的风险描述。
- 只根据 diff 的实际内容判断，不要靠函数名、文件名或没展示出来的代码猜。
- 跨文件的问题要看出来：调用方和被调方在同一个 diff 里，签名和实参对不上必须指出。
- 如果某处看起来可疑、但按这份 diff 判断其实没问题，放进 not_issues 说明理由，不要凑数报成 issue。
待评审的 PR diff：
diff --git a/src/api/orders.py b/src/api/orders.py
@@ -12,6 +12,21 @@ from src.service.order_service import OrderService
+@app.route("/api/orders/search")
+def search_orders():
+ """按状态和关键词查订单，供运营后台用"""
+ status = request.args.get("status", "paid")
+ kw = request.args.get("kw", "")
+ tenant = g.current_tenant_id
+ sql = (
+ f"SELECT id, uid, amount, status FROM orders "
+ f"WHERE tenant_id = {tenant} AND status = '{status}' "
+ f"AND title LIKE '%{kw}%' ORDER BY created_at DESC LIMIT 200"
+ )
+ rows = db.execute(sql).fetchall()
+ tmp1 = [dict(r) for r in rows]
+ return jsonify(tmp1)
+
diff --git a/src/service/order_service.py b/src/service/order_service.py
@@ -30,10 +30,25 @@ class OrderService:
- def list_by_user(self, uid):
- return db.query(Order).filter(Order.uid == uid).all()
+ def list_by_user(self, uid):
+ key = f"order:list:{uid}"
+ hit = self.cache.get(key)
+ if hit is not None:
+ return json.loads(hit)
+ rows = (db.query(Order)
+ .filter(Order.uid == uid,
+ Order.tenant_id == g.current_tenant_id)
+ .all())
+ self.cache.set(key, json.dumps([r.to_dict() for r in rows]), ex=300)
+ return rows
+
+
+def build_report(month: int):
+ """出指定月份的订单月报"""
+ if month < 1 or month > 12:
+ raise ValueError("month out of range")
+ return _aggregate(month)
diff --git a/src/api/report.py b/src/api/report.py
@@ -5,6 +5,11 @@ from src.service.order_service import build_report
+@app.route("/api/orders/report")
+def order_report():
+ month = request.args["month"]
+ data = build_report(month)
+ return jsonify(data)
diff --git a/src/service/payment.py b/src/service/payment.py
@@ -58,10 +58,16 @@ class PaymentService:
def settle(self, order_id):
order = self._load(order_id)
- self.gateway.charge(order.uid, order.amount)
- order.status = "paid"
- db.commit()
+ try:
+ self.gateway.charge(order.uid, order.amount)
+ except Exception:
+ pass
+ order.status = "paid"
+ db.commit()
+ self.notifier.push(order.uid, "支付成功")
diff --git a/src/util/etag.py b/src/util/etag.py
@@ -0,0 +1,9 @@
+import hashlib
+
+
+def make_etag(payload: bytes) -> str:
+ """给响应体算一个 ETag，只用于协商缓存，不涉及任何凭据"""
+ return hashlib.md5(payload).hexdigest()
diff --git a/src/api/handler.py b/src/api/handler.py
@@ -33,8 +33,7 @@ def handle_request(req):
ctx = build_context(req)
- print("DEBUG ctx:", ctx)
- resp = dispatch(ctx, timeout=cfg.timeout)
+ resp = dispatch(ctx, timeout=300)
return resp
