bitsrun
bitsrun copied to clipboard
支持不同的认证网关 Support custom srun authentication portal via auth_url
TLDR: 支持自定义 srun 认证 portal,兼容外校/不同认证地址。
修改内容:
- 在 user.py 相关逻辑中增加了 auth_url 参数,所有认证相关请求均可自定义 portal 地址。
- 对所有的CLI命令(如 login/logout/status)均支持 --auth_url 参数,用户可通过命令行指定认证 portal。
- README 增加了“Srun at other campuses”部分,说明如何通过 auth_url 字段或 CLI 参数兼容其他学校或自定义 portal。
需要讨论+可能可以改进的地方:
- 我在status和log{in/out}分别添加了auth_url的命令行参数,是否有更优雅的方法
- 需要review是否影响了不添加auth_url时的代码表现
Ruff把参数拆成了多行,是因为太长了吗🤔
我在status和log{in/out}分别添加了auth_url的命令行参数,是否有更优雅的方法
这种不敏感又不常改变的参数,可能用环境变量比较好?click 内置支持。不过已经有配置文件了,环境变量倒是也没太大优势。
Ruff把参数拆成了多行,是因为太长了吗🤔
估计是吧。可能参数多了后,分多行 git diff 比较清楚……