fetch_attachment does not consider '_rev_'

未关闭
#192 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

还没有人认领这个 Issue。

评估

难度
2/5
预计耗时
1-3 小时
新手友好度
48/100
Issue 类型
缺陷
描述清晰度
描述清楚
活跃度
停滞
技术栈
python
领域
databases

调研方向

从 client.py 中的 fetch_attachment 开始,检查请求附件时如何使用 document 参数。使用包含 _rev 的文档复现该情况,然后验证附件请求是否针对该 revision,并保留预期的冲突行为。

由索引模型根据 Issue 内容生成。

描述

As far I can see (and test) fetch_attachment does not consider the _rev field when it is passed in the "doc".

This can cause problems: we are going to get always the latest document, avoiding any conflict.

Here my patch:

$ diff client.py ~/sources/couchdbkit-0.6.5/couchdbkit/client.py 
855d854
<             doc = {}
863c862
<         resp = self.res(docid).get(name, headers=headers, rev=doc.get('_rev'))

---
>         resp = self.res(docid).get(name, headers=headers)
主要语言
Python
星标
261
派生
90
PR 合并指标
30 天内没有已合并 PR

环境准备

这个项目没有提供开发容器、Dockerfile 或贡献指南,环境需要你自己搭建:先看它的 README,通用步骤见我们的新手贡献指南。

从这里开始

  1. 先读完整个 Issue,再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库,在一个分支上完成修改。
  4. 提交 Pull Request,并在描述里引用这个 Issue 编号。

benoitc/couchdbkit 的其他 Issue

查看 benoitc/couchdbkit 的全部 Issue

相似的 Issue

更多 Python Issue

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。