Update deprecated ghcr.io image reference to public.ecr.aws - #27
Update deprecated ghcr.io image reference to public.ecr.aws#27luisina-santos wants to merge 1 commit into
Conversation
README.md referenced this connector's image via ghcr.io/conductorone; container images are now published to public.ecr.aws/conductorone instead. Only this connector's own image reference was updated. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
| ``` | ||
| docker run --rm -v $(pwd):/out -e BATON_TOKEN=githubAccessToken ghcr.io/conductorone/baton-github-enterprise:latest -f "/out/sync.c1z" | ||
| docker run --rm -v $(pwd):/out -e BATON_TOKEN=githubAccessToken public.ecr.aws/conductorone/baton-github-enterprise:latest -f "/out/sync.c1z" | ||
| docker run --rm -v $(pwd):/out ghcr.io/conductorone/baton:latest -f "/out/sync.c1z" resources |
There was a problem hiding this comment.
🟡 Suggestion: The adjacent line still pulls the baton CLI from the deprecated ghcr.io registry. If ConductorOne images have moved to public.ecr.aws org-wide (per CXH-2418), this line will break for users the same way the one above it would have. Confidence: medium — intentionally out of scope per the PR description, but worth confirming public.ecr.aws/conductorone/baton:latest exists and updating it here too.
| docker run --rm -v $(pwd):/out ghcr.io/conductorone/baton:latest -f "/out/sync.c1z" resources | |
| docker run --rm -v $(pwd):/out public.ecr.aws/conductorone/baton:latest -f "/out/sync.c1z" resources |
Connector PR Review: Update deprecated ghcr.io image reference to public.ecr.awsBlocking Issues: 0 | Suggestions: 1 | Threads Resolved: 0 Review SummaryThe full PR diff was scanned for security and correctness: it is a single-line README change swapping this connector's Docker image reference from Security IssuesNone found. Correctness IssuesNone found. Suggestions
Prompt for AI agents |
Summary
ghcr.io/conductorone/baton-github-enterprise:latest; container images are now published topublic.ecr.aws/conductorone/baton-github-enterprise:latestinstead.Test plan
public.ecr.aws/conductorone/baton-github-enterprise:latestresolves (manifest exists) before opening this PR.🤖 Generated with Claude Code