Skip to content

Use shared git output sanitizer in prefect-bitbucket#22199

Merged
zzstoatzz merged 1 commit into
mainfrom
codex/adopt-sanitizer-prefect-bitbucket
Jun 12, 2026
Merged

Use shared git output sanitizer in prefect-bitbucket#22199
zzstoatzz merged 1 commit into
mainfrom
codex/adopt-sanitizer-prefect-bitbucket

Conversation

@zzstoatzz

@zzstoatzz zzstoatzz commented Jun 2, 2026

Copy link
Copy Markdown
Collaborator

Depends on Prefect >=3.7.4, which includes prefect._internal.urls.strip_auth_from_urls_in_text from #22196.

This is a draft adoption PR for prefect-bitbucket.

Changes

  • sanitize Bitbucket repository block async and sync clone stderr with the shared core helper
  • pass configured Bitbucket token material as extra redaction secrets so token echoes outside URL-shaped stderr are also redacted
  • add regression coverage for a malformed credential URL such as https://dev:p@ss:word#1@bitbucket.org/...
  • bump the package's Prefect lower bound to prefect>=3.7.4

Validation

  • uv run pytest tests/test_repository.py -q -k redacts_token_in_clone_error from src/integrations/prefect-bitbucket
  • uv run ruff check src/integrations/prefect-bitbucket/prefect_bitbucket/repository.py src/integrations/prefect-bitbucket/tests/test_repository.py from repo root
  • uv run ruff format --check src/integrations/prefect-bitbucket/prefect_bitbucket/repository.py src/integrations/prefect-bitbucket/tests/test_repository.py from repo root

Base automatically changed from codex/core-sanitized-git-clone-stderr to main June 4, 2026 16:51
@zzstoatzz zzstoatzz force-pushed the codex/adopt-sanitizer-prefect-bitbucket branch from f2bbe1e to 1603c4c Compare June 12, 2026 18:32
@zzstoatzz zzstoatzz marked this pull request as ready for review June 12, 2026 18:36
@zzstoatzz zzstoatzz merged commit 6f66e8d into main Jun 12, 2026
20 checks passed
@zzstoatzz zzstoatzz deleted the codex/adopt-sanitizer-prefect-bitbucket branch June 12, 2026 19:10
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.

2 participants