Leo Q

Results 239 comments of Leo Q

有条件的话开一下debug,看一下详情,csrf更多的还是django和代理的问题,你需要排查一整条链路上的代理,看最后给到django app 的请求是怎么样的,这样来看哪里出了问题。

这个无关, 建议还是多看一下CSRF 的相关文档, 这个更多的还是 django 问题而不是 archery 问题, 因为我也没有环境去复现, 所以我暂时没法帮到你

可以用环境变量去设置全局的市区,也欢迎提交任何的PR

达梦和mysql 和oracle 都可以兼容,你新做的模块新增了什么功能吗?

不好意思, 我可能回复的太武断了, 我认知里, 达梦是支持使用 mysqlclient 进行连接的, 我也不太清楚这个认知是否正确, 我这个认知是否有一定的误区? 如果我说错了欢迎指正. 如果达梦支持使用 mysqlclient 进行连接, 那么直接使用 mysql engine 即可. 如果达梦仅支持达梦自己的sdk 进行连接, 那么非常欢迎你的 PR. 另外PR 当前有一个问题, 就是将`备份功能优化`和`达梦数据库` 支持混在了一起, 我个人认为, 达梦数据库支持是一个相对常规的 pr, 这个 pr 我想接受起来会比较简单,...

你当前的备份逻辑还是有一些值得探讨的地方的, 我还是比较建议你将其抽离出来单独提交, 比如你当前的设计, 备份的回滚语句是用 json 方式存储在数据库中的, 那么这个数据是否会太大? 是否会有效率问题? 这都需要再探讨一下, 如果我们两个功能一起来讨论, 势必会增大复杂性, 并且拖延你达梦数据库适配的进度.

Thanks for your pr! Please add tests for the change so it can be safely merged.

确实,这一块是重构时显示成了ID,还没时间修,应该在这里重新取一下Name然后写进去就行了,有时间的话也欢迎提PR

https://github.com/hhyo/Archery/blob/58c01452a8bdf4c463f93bd97efbdd8cca9cd572/sql/utils/workflow_audit.py#L504 这里,有修改建议可以直接提pr