Skip to content

test: lock token-bound database routing priority - #158

Merged
FreeOnePlus merged 1 commit into
apache:masterfrom
FreeOnePlus:test/issue-64-token-routing
Jul 30, 2026
Merged

test: lock token-bound database routing priority#158
FreeOnePlus merged 1 commit into
apache:masterfrom
FreeOnePlus:test/issue-64-token-routing

Conversation

@FreeOnePlus

Copy link
Copy Markdown
Member

Fixes #64

Root cause

Token-bound database routing is already implemented in the current server, but there was no regression test covering the important case where a valid global Doris account is configured at the same time. That left room for a future regression to silently route an authenticated token through the global high-privilege account.

Change

  • Add a regression test with both a valid global account and a token-bound account.
  • Assert the dedicated pool is created from the token database configuration.
  • Assert the original and active global configurations remain unchanged.

Verification

  • uv run pytest --no-cov -q test/utils/test_doris_user_pool_manager.py — 15 passed
  • uv run ruff check test/utils/test_doris_user_pool_manager.py — passed
  • Full repository tests and coverage run in CI.

@FreeOnePlus
FreeOnePlus merged commit 0ae690c into apache:master Jul 30, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

能否支持不同的token进来,使用对应的账号进行数据库访问

1 participant