Get rid of Class variable storage and move to instance storage

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

还没有人认领这个 Issue。

评估

难度
4/5
预计耗时
3-5 天
新手友好度
25/100
Issue 类型
重构
描述清晰度
需要澄清
活跃度
停滞
技术栈
python
领域
api, backend

调研方向

首先跟踪 user、password 和 site_uri 在哪里存储,以及 REST 请求如何访问它们,然后检查如何为不同的 Shopify 环境创建和选择 session。完成的标准是 production 和 QA session 可以在内存中共存,并且并发请求不再相互覆盖配置;为这两种情况添加或运行 coverage。

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

描述

feature request

I'm building an admin/monitoring server that handles requests from both out production and qa Shopify store.
In the current setup, the user/password/site_uri are stored at the class level, so I can't create a prod session and a qa session and store them in memory.
For every request to the REST server I have to clear the current class variables and set for requested environment.

Additionally, this is not thread safe. If two REST requests arrive at the same time for different endpoints, they will clobber each other.

主要语言
Python
星标
1.4k
派生
388
平均合并
5 小时 39 分钟
30 天内合并 PR
1

贡献指南

打开贡献指南

从这里开始

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

Shopify/shopify_python_api 的其他 Issue

查看 Shopify/shopify_python_api 的全部 Issue

相似的 Issue

更多 Python Issue

把新 issue 发到你的邮箱

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