Skip to content

Commit 5b02de0

Browse files
caseylockerclaude
andcommitted
fix: Add IDropboxMaterializerApi to provides(), restrict retries to GET
- Register in deferred provider's provides() so DI resolution works - Limit GuzzleRetryMiddleware to GET requests only (POST endpoints are idempotent but no reason to retry them on 429/503) Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent f18a6a1 commit 5b02de0

2 files changed

Lines changed: 4 additions & 1 deletion

File tree

app/Services/Apis/DropboxMaterializerApi.php

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,9 @@ final class DropboxMaterializerApi implements IDropboxMaterializerApi
3737
public function __construct()
3838
{
3939
$stack = HandlerStack::create();
40-
$stack->push(GuzzleRetryMiddleware::factory());
40+
$stack->push(GuzzleRetryMiddleware::factory([
41+
'retry_on_methods' => ['GET'],
42+
]));
4143

4244
$this->client = new Client([
4345
'handler' => $stack,

app/Services/BaseServicesProvider.php

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -199,6 +199,7 @@ public function provides()
199199
IPushNotificationApi::class,
200200
IGeoCodingAPI::class,
201201
IExternalUserApi::class,
202+
IDropboxMaterializerApi::class,
202203
IFolderService::class,
203204
ILockManagerService::class,
204205
IPasswordlessAPI::class,

0 commit comments

Comments
 (0)