2 Commits (dev)

Author	SHA1	Message	Date
Your Name	7b11b72e1d	test(federation): run AP delivery tests in production env ... ActivityPubDeliveryService::pool() and queueDelivery() only perform HTTP delivery when app()->environment('production'); outside production they log and return. The pool/job delivery tests were written before that guard was added (commit 868e09b64) and never forced production, so no requests were made and every assertSent/assertSentCount failed (6 failing tests). Wrap the delivery-exercising tests in a forged production environment (restored afterwards) and seed the public-ips DNS cache plus an empty banned-domains cache so validateDestination() passes without a real DNS lookup or DB query. Seed after factory creation, since the lazy DB refresh can flush the cache store. Also fix a by-value capture bug in the onError assertions: the $errors array was captured through an arrow-fn wrapper, so the inner callback's by-reference mutations never reached the assertion. Use a regular closure. No production code changed; the production delivery gate is intended behavior.	2 weeks ago
Your Name	9c9e2a5a22	refactor: extract shared ActivityPub pool delivery into ActivityPubDeliveryService ... Add ActivityPubDeliveryService::pool() using Laravel's Http::pool() to consolidate the duplicated delivery pattern found across 10 jobs. Updated jobs: - StatusActivityPubDeliver - StatusDelete - StatusLocalUpdateActivityPubDeliverPipeline - FanoutDeletePipeline - SharePipeline - UndoSharePipeline - StoryFanout - StoryExpire - StoryDelete - ProfileMigrationDeliverMoveActivityPipeline The shared method accepts a Profile (sender), audience (inbox URLs), and activity (payload array), handling signing, user-agent, timeout, and concurrency in one place. No direct Guzzle usage remains in app/Jobs/.	4 weeks ago