Skip to content

Fix webhooks error message#11422

Open
eldadfux wants to merge 1 commit into1.8.xfrom
fix-webhooks-error-message
Open

Fix webhooks error message#11422
eldadfux wants to merge 1 commit into1.8.xfrom
fix-webhooks-error-message

Conversation

@eldadfux
Copy link
Member

And adding telemetry instead

@eldadfux eldadfux changed the base branch from main to 1.8.x February 27, 2026 05:24
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 27, 2026

Caution

Review failed

Failed to post review comments

📝 Walkthrough

Walkthrough

This pull request introduces a comprehensive refactor of Appwrite's routing and API infrastructure, with updates to documentation, configuration, and database schemas. Changes include migrations from App:: to Http:: routing across API controllers, removal of endpoints related to avatars, health, console, storage, and teams, introduction of new error mappings and scopes organization, updates to environment variables supporting multiple database adapters (MariaDB, MongoDB, PostgreSQL), and configuration of new features like deployment retention and site specifications. GitHub Actions workflows are upgraded to newer action versions, and Docker infrastructure is updated to support multiple database backends and new worker types.

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~75 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 62.50% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The PR title 'Fix webhooks error message' is directly related to a subset of changes in the changeset, specifically webhook-related fixes mentioned in the commit messages.
Description check ✅ Passed The PR description 'And adding telemetry instead' is related to the changeset, as commit messages indicate 'Silence webhook exceptions, adding telemetry' and webhook-related improvements.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix-webhooks-error-message

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link

Security Scan Results for PR

Docker Image Scan Results

Package Version Vulnerability Severity
libecpg 18.1-r0 CVE-2026-2004 HIGH
libecpg 18.1-r0 CVE-2026-2005 HIGH
libecpg 18.1-r0 CVE-2026-2006 HIGH
libecpg 18.1-r0 CVE-2026-2007 HIGH
libecpg-dev 18.1-r0 CVE-2026-2004 HIGH
libecpg-dev 18.1-r0 CVE-2026-2005 HIGH
libecpg-dev 18.1-r0 CVE-2026-2006 HIGH
libecpg-dev 18.1-r0 CVE-2026-2007 HIGH
libheif 1.20.2-r1 CVE-2025-68431 HIGH
libpng 1.6.54-r0 CVE-2026-25646 HIGH
libpng-dev 1.6.54-r0 CVE-2026-25646 HIGH
libpq 18.1-r0 CVE-2026-2004 HIGH
libpq 18.1-r0 CVE-2026-2005 HIGH
libpq 18.1-r0 CVE-2026-2006 HIGH
libpq 18.1-r0 CVE-2026-2007 HIGH
libpq-dev 18.1-r0 CVE-2026-2004 HIGH
libpq-dev 18.1-r0 CVE-2026-2005 HIGH
libpq-dev 18.1-r0 CVE-2026-2006 HIGH
libpq-dev 18.1-r0 CVE-2026-2007 HIGH
postgresql18-dev 18.1-r0 CVE-2026-2004 HIGH
postgresql18-dev 18.1-r0 CVE-2026-2005 HIGH
postgresql18-dev 18.1-r0 CVE-2026-2006 HIGH
postgresql18-dev 18.1-r0 CVE-2026-2007 HIGH

Source Code Scan Results

🎉 No vulnerabilities found!

@github-actions
Copy link

✨ Benchmark results

  • Requests per second: 2,335
  • Requests with 200 status code: 420,317
  • P99 latency: 0.074104272

⚡ Benchmark Comparison

Metric This PR Latest version
RPS 2,335 1,379
200 420,317 248,186
P99 0.074104272 0.156959834

@github-actions
Copy link

github-actions bot commented Feb 27, 2026

🔄 PHP-Retry Summary

Flaky tests detected across commits:

Commit a5c37db - 6 flaky tests
Test Retries Total Time Details
LegacyConsoleClientTest::testUpdateAttributeEnum 1 240.50s Logs
LegacyCustomServerTest::testCreatedBefore 1 240.31s Logs
TablesDBConsoleClientTest::testAttributeResponseModels 1 243.78s Logs
TablesDBTransactionsConsoleClientTest::testCreateDocument 1 240.94s Logs
UsageTest::testDatabaseStatsCollectionsAPI 1 10.19s Logs
RealtimeConsoleClientTest::testDeleteIndexCollectionsAPI 1 120.13s Logs

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.

1 participant