bitsrun icon indicating copy to clipboard operation
bitsrun copied to clipboard

支持不同的认证网关 Support custom srun authentication portal via auth_url

Open panda361 opened this issue 4 months ago • 3 comments

TLDR: 支持自定义 srun 认证 portal,兼容外校/不同认证地址。

修改内容:

  1. 在 user.py 相关逻辑中增加了 auth_url 参数,所有认证相关请求均可自定义 portal 地址。
  2. 对所有的CLI命令(如 login/logout/status)均支持 --auth_url 参数,用户可通过命令行指定认证 portal。
  3. README 增加了“Srun at other campuses”部分,说明如何通过 auth_url 字段或 CLI 参数兼容其他学校或自定义 portal。

需要讨论+可能可以改进的地方:

  1. 我在status和log{in/out}分别添加了auth_url的命令行参数,是否有更优雅的方法
  2. 需要review是否影响了不添加auth_url时的代码表现

panda361 avatar Sep 16 '25 13:09 panda361

Ruff把参数拆成了多行,是因为太长了吗🤔

panda361 avatar Sep 16 '25 13:09 panda361

我在status和log{in/out}分别添加了auth_url的命令行参数,是否有更优雅的方法

这种不敏感又不常改变的参数,可能用环境变量比较好?click 内置支持。不过已经有配置文件了,环境变量倒是也没太大优势。

Ruff把参数拆成了多行,是因为太长了吗🤔

估计是吧。可能参数多了后,分多行 git diff 比较清楚……

YDX-2147483647 avatar Sep 16 '25 13:09 YDX-2147483647

这种不敏感又不常改变的参数,可能用环境变量比较好?click 内置支持。不过已经有配置文件了,环境变量倒是也没太大优势。

我研究研究

panda361 avatar Sep 16 '25 13:09 panda361