MTM Improve app by leveraging more Laravel code #111

Merged

jonny merged 4 commits from develop into main 2026-08-01 20:37:16 +02:00

Conversation 0 Commits 4 Files changed 36 +1331 -1431

jonny commented 2026-08-01 20:37:10 +02:00

Owner

Copy link

Specifically we are now using Laravel’s fluent Images API, as well as the Http for requests and tests and forcing the prod link to be https:// links.

Specifically we are now using Laravel’s fluent Images API, as well as the `Http` for requests and tests and forcing the prod link to be `https://` links.

jonny added 4 commits 2026-08-01 20:37:10 +02:00

Migrate outbound HTTP calls from Guzzle to the Http facade ... e08763c526

Replaces every raw GuzzleHttp\Client usage (Nominatim, web.archive.org,
webmention fetch/discover/send, Bridgy syndication, IndieAuth client_id
lookup, contact avatar/h-card fetch, profile image download, and the
CloudConvert screenshot pipeline) with Illuminate\Support\Facades\Http,
and enables Http::preventStrayRequests() globally in tests so any
un-faked outbound call now fails loudly instead of silently hitting the
network.

The CloudConvert retry-until-finished middleware in AppServiceProvider
is replaced by a plain polling loop in SaveScreenshot, which also fixes
a latent bug where the old middleware decoded a response object instead
of its body. Bridgy syndication jobs keep their tries=1/no-retry
semantics unchanged to avoid duplicate publishes. Guzzle's PSR-7 helpers
(Header, UriResolver, Utils) stay in SendWebMentions since Http has no
equivalent for them.

Every affected test file's Guzzle MockHandler/HandlerStack boilerplate
is replaced with Http::fake()/Http::sequence().

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01P8j7cJhCiYDsGUgB7obKNQ

Update dependencies 2eab5749ea

Force HTTPS URL generation in production ... 287520ad7b

Uses Laravel's built-in URL::forceHttps() so route(), url(), and asset()
always generate https:// links in production, without affecting local dev.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01RP5TeZbk9KS754xuJMobjE

Adopt Laravel's Image facade for media processing, upgrade Intervention to v4 ... 31c49ac3fc

Laravel 13's Image facade wraps Intervention Image v4 internally, so
switching our upload width probe and resize job/command to it required
bumping intervention/image ^3 -> ^4 (and its intervention/gif ^5
dependency). Removes our own ImageManager container binding and
config/image.php in favour of Laravel's built-in driver resolution.

Also fixes a latent filename mismatch in ProcessMediaJobTest that Pint's
stricter typing on the new Image API turned into a hard TypeError, and
tidies config/flare.php to use imported class names instead of FQCNs.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_018Sorsgn85nw7uQyRMNvzyD

jonny merged commit db2c7b19ed into main 2026-08-01 20:37:16 +02:00

jonny referenced this pull request from a commit 2026-08-01 20:37:17 +02:00

Merge pull request 'MTM Improve app by leveraging more Laravel code' (#111) from develop into main

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels

No items

No labels

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference

jonny/jonnybarnes.uk!111

No description provided.