Skip to content

Report artifact source token path#47

Closed
shiny-code-bot wants to merge 2 commits into
mainfrom
fix/source-token-diagnostics
Closed

Report artifact source token path#47
shiny-code-bot wants to merge 2 commits into
mainfrom
fix/source-token-diagnostics

Conversation

@shiny-code-bot
Copy link
Copy Markdown
Collaborator

Summary

  • report which non-secret source-token path artifact selector resolution selected
  • include whether git auth header configuration was enabled for selector resolution
  • keep token values out of logs

Tests

  • uv run python -m unittest tests.test_runtime.RuntimeCommandTests.test_resolve_artifact_runtime_source_refs_uses_environment_token_fallback tests.test_runtime.RuntimeCommandTests.test_resolve_artifact_runtime_source_refs_uses_dedicated_source_token_env tests.test_runtime.RuntimeCommandTests.test_resolve_artifact_runtime_source_refs_supports_ci_source_token_env tests.test_runtime.RuntimeCommandTests.test_resolve_artifact_runtime_source_refs_prefers_runtime_github_token tests.test_runtime.RuntimeCommandTests.test_resolve_source_repository_ref_to_git_sha_rejects_missing_remote_ref
  • uv run ruff check odoo_devkit/local_runtime.py tests/test_runtime.py
  • uv run ruff format --check odoo_devkit/local_runtime.py tests/test_runtime.py

"GIT_CONFIG_VALUE_0": f"AUTHORIZATION: basic {encoded_auth}",
}
)
print(f"artifact_source_token_source={github_token_source} auth_header_configured={str(normalized_token is not None).lower()}")
@shiny-code-bot
Copy link
Copy Markdown
Collaborator Author

Closing this diagnostic-only branch: CodeQL correctly rejected logging around token path metadata, so I am moving the probe into the tenant workflow preflight without printing token-derived values.

@shiny-code-bot shiny-code-bot deleted the fix/source-token-diagnostics branch May 28, 2026 23:15
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.

3 participants