Skip to content

fix(webhook): restore GitLab source deployments - #11284

Merged
peaklabs-dev merged 2 commits into
coollabsio:mainfrom
drewmt:fix/gitlab-webhook-cuid2
Aug 18, 2026
Merged

fix(webhook): restore GitLab source deployments#11284
peaklabs-dev merged 2 commits into
coollabsio:mainfrom
drewmt:fix/gitlab-webhook-cuid2

Conversation

@drewmt

@drewmt drewmt commented Aug 14, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Changes

  • Restore the correct CUID2 namespace for normal GitLab source webhooks, so push and merge-request events can generate deployment identifiers instead of returning HTTP 500.
  • Add feature coverage proving that a valid GitLab push webhook reaches the deployment queue with the received commit.

Issues

Category

  • Bug fix
  • Improvement
  • New feature
  • Adding new one click service
  • Fixing or updating existing one click service

Preview

Not applicable; this is a backend webhook fix.

AI Assistance

  • AI was NOT used to create this PR
  • AI was used (please describe below)

If AI was used:

  • Tools used: Development assistance
  • How extensively: Used during implementation and test preparation; the resulting two-file diff was inspected and validated locally.

Testing

  • Confirmed the new feature test fails before the fix with Class "App\\Http\\Controllers\\Webhook\\Cuid2" not found and HTTP 500.
  • php artisan test --compact tests/Feature/GitlabWebhookTest.php tests/Feature/GitlabAppWebhookTokenEncryptionTest.php tests/Feature/QueueApplicationDeploymentCommitTest.php — 7 passed, 21 assertions.
  • GitLab-specific backend suite — 64 passed, 166 assertions.
  • Pint, PHP syntax checks, composer validate --strict, and git diff --check pass.
  • Three unrelated GitLab source UI-copy assertions fail unchanged on a clean next baseline.

Contributor Agreement

Important

  • I have read and understood the contributor guidelines. If I have failed to follow any guideline, I understand that this PR may be closed without review.
  • I have searched existing issues and pull requests (including closed ones) to ensure this isn't a duplicate.
  • I have tested all the changes thoroughly with a local development instance of Coolify and I am confident that they will work as expected when a maintainer tests them.

@github-actions
github-actions Bot changed the base branch from next to main August 14, 2026 15:26
@github-actions

github-actions Bot commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Based on the selected category, this PR's base branch was automatically changed from next to main.

Bug fixes and one-click service changes target main.

If you believe this is incorrect, please let a maintainer know.

peaklabs-dev
peaklabs-dev approved these changes Aug 18, 2026
View reviewed changes

@peaklabs-dev peaklabs-dev left a comment

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.

Thanks for your PR!

@peaklabs-dev
peaklabs-dev merged commit 264cb8e into coollabsio:main Aug 18, 2026
1 check passed
@github-actions github-actions Bot locked as resolved and limited conversation to collaborators Sep 18, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

@peaklabs-dev peaklabs-dev peaklabs-dev approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[Bug]: GitLab source push webhook fails with 500 because Cuid2 is not imported

2 participants

@drewmt @peaklabs-dev