Skip to content

fix(ci): report website rebuild dispatch failures - #103

Merged
ai merged 1 commit into
logux:mainfrom
arestov:fix/site-rebuild-trigger
Oct 15, 2026
Merged

ai merged 1 commit into
logux:mainfrom
arestov:fix/site-rebuild-trigger

Conversation

@arestov

@arestov arestov commented Oct 15, 2026

Copy link
Copy Markdown
Contributor

Make website rebuild requests fail on HTTP errors or missing credentials instead of silently succeeding. Pass credentials through environment variables, use standard GitHub API headers, and bound request time.

Requires the website receiver in logux/logux.org#64; merge that PR first. This keeps the existing push-to-main trigger and dispatch credentials.

Validation: actionlint and six local Bash/curl scenarios (HTTP 204/401/403/500 and missing username/token).

Comment thread .github/workflows/deploy.yml Outdated
permissions: {}
jobs:
deploy:
if: github.event.deleted == false

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why we need this check?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(perf optimization to skip rebuild on tag/branch removing, but we don't need it with branches: -main)

@arestov
arestov force-pushed the fix/site-rebuild-trigger branch from 5bc11ff to cbf3005 Compare October 15, 2026 15:26
@ai
ai merged commit 74c5257 into logux:main Oct 15, 2026
1 check 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.

2 participants