EnAccess / EnAccess/OpenPAYGO-python
[Bug]: Inconsistent `value` behaviour between `generate_token` and `decode_token` for `DISABLE_PAYG` and `COUNTER_SYNC`
- 主要语言
- Python
- 星标
- 15
- 派生
- 15
- PR 合并指标
- 30 天内没有已合并 PR
描述
### Preflight Checklist
- [x] I have read the [Contributing Guidelines](https://github.com/EnAccess/OpenPAYGO-python/blob/main/CONTRIBUTING.md) for this project, if it exists.
- [x] I agree to follow the [Code of Conduct](https://github.com/EnAccess/OpenPAYGO-python/blob/main/CODE_OF_CONDUCT.md) that this project adheres to.
- [x] I have searched the [issue tracker](https://github.com/EnAccess/OpenPAYGO-python/issues) for a feature request that matches the one I want to file, without success.
### Expected Behavior
Token value and Token Type have a 1:1 relation for `DISABLE_PAYG` and `COUNTER_SYNC`. As a user I would expect `generate_token` and `decode_token` to have the same interface in terms of how `DISABLE_PAYG` and `COUNTER_SYNC` are specified.
### Actual Behavior
In **generate_token** only `token_type` must be specified. Providing a non-`None` value yields:
```sh
ValueError: A value is not allowed for this token type.
```
In **decode_token** both `token_type` and `token_value` are returned.
### Steps to reproduce
```python
# OpenPAYGO-python
from openpaygo import OpenPAYGOTokenEncoder, TokenType
encoder = OpenPAYGOTokenEncoder()
new_count, final_token = encoder.generate_token(
secret_key="bc41ec9530f6dac86b1a29ab82edc5fb",
count=4,
value=998,
token_type=TokenType.DISABLE_PAYG,
starting_code=516959010,
restricted_digit_set=False,
extended_token=False,
)
print(f"{new_count=}")
print(f"{final_token=}")
```
### What operating system(s) are you using?
macOS
### Additional Information
**Suggested solutions:**
Change `generate_token` to allow a token value that matches the provided token type. If none is given, the token value is derived from token_type. If a token type is provided that does not match the token value throw an error explaining the problem.
**Alternative solution:**
Change `decode_token` to not return token values for `DISABLE_PAYG` and `COUNTER_SYNC`
贡献指南
调研方向
首先在 OpenPAYGO-python 包中找到 generate_token 和 decode_token 入口点,并复现所提供的 DISABLE_PAYG 示例。比较 token_type 和 token_value 的接受方式及返回方式,然后从建议的解决方案中选择预期的一致契约;当两个接口都以一致的方式处理这些 token 类型时,该 issue 即完成。
由索引模型根据 Issue 内容生成。
评估
- 技术栈
- python
- 领域
- api
- Issue 类型
- 缺陷
- 难度
- 3/5
- 预计耗时
- 1-2 天
- 活跃度
- 停滞
- 描述清晰度
- 基本清楚
- 新手友好度
- 45/100