Skip to content

fix(selfhost): unset POSTHOG_CLI_HOST when blank, don't pass empty #8622

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
JSONbored merged 2 commits into from
Jul 25, 2026
+5 −0
Merged

fix(selfhost): unset POSTHOG_CLI_HOST when blank, don't pass empty #8622

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
407b0be
fix(selfhost): unset POSTHOG_CLI_HOST when blank, don't pass empty
JSONbored Jul 25, 2026
35aca7b
fix(selfhost): reword comment to stop tripping the SENTRY_ drift guard
JSONbored Jul 25, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions .github/workflows/release-selfhost.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -166,6 +166,10 @@ jobs:
set -euo pipefail
test -n "$POSTHOG_CLI_API_KEY"
test -n "$POSTHOG_CLI_PROJECT_ID"
# posthog-cli treats a PRESENT-but-empty POSTHOG_CLI_HOST as an explicit (invalid) URL, not as
# "unset, use the default" -- unset it outright when blank, same guard this pipeline's old
# Sentry-org-URL override once needed for the identical reason.
if [ -z "${POSTHOG_CLI_HOST:-}" ]; then unset POSTHOG_CLI_HOST; fi
# No separate "create release" step -- PostHog release metadata is a byproduct of the inject/
# upload calls below, unlike Sentry's releases/commits/deploys/finalize lifecycle this replaces.
npx -y "$POSTHOG_CLI_PACKAGE" sourcemap inject --directory dist --release-version "$POSTHOG_RELEASE"
Expand Down Expand Up @@ -259,6 +263,7 @@ jobs:
POSTHOG_RELEASE: ${{ steps.version.outputs.release }}
run: |
set -euo pipefail
if [ -z "${POSTHOG_CLI_HOST:-}" ]; then unset POSTHOG_CLI_HOST; fi
# PostHog's release-read path can lag briefly behind the upload write above.
# review-enrichment/src/upload-sourcemaps.ts's runReleaseValidation() already retry-polls this
# same script for REES's own deploy path -- mirror that here instead of failing the whole
Expand Down