Appwrite/appwrite
1
0
Fork 0
mirror of https://github.com/appwrite/appwrite.git synced 2026-05-26 13:51:13 +00:00
Code Issues Packages Projects Releases Wiki Activity

Sanitize bubbled message for non-typed exceptions on migration document

Browse Source
This commit is contained in:
Prem Palanisamy
2026-05-07 13:45:57 +01:00
parent 81e57ea30e
commit 3676af925c
1 changed files with 18 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/Appwrite/Platform/Workers/Migrations.php
+18 -3
View File
@@ -572,15 +572,30 @@ class Migrations extends Action
$destinationErrors = $destination?->getErrors() ?? [];
if ($caughtError !== null) {
$bubbled = $caughtError instanceof MigrationException
? $caughtError
: new MigrationException(
if ($caughtError instanceof MigrationException) {
// library-thrown, message constructed by us
$bubbled = $caughtError;
} elseif ($caughtError instanceof Exception) {
// typed AppwriteException — message comes from the curated registry
$bubbled = new MigrationException(
resourceName: '',
resourceGroup: '',
message: $caughtError->getMessage(),
code: $caughtError->getCode(),
previous: $caughtError,
);
} else {
// unknown throwable — raw message may embed internal hostnames,
// DSNs, tokens, etc. Replace with a generic user-facing string;
// the original is preserved on `previous:` for Sentry.
$bubbled = new MigrationException(
resourceName: '',
resourceGroup: '',
message: 'Migration failed due to an unexpected error.',
code: $caughtError->getCode() ?: 500,
previous: $caughtError,
);
}
$destinationErrors[] = $bubbled;
}