Fix CustomEmoji, properly handle shortcode updates and delete old copy in case the extension changes

This commit is contained in:
Daniel Supernault 2022-12-24 01:07:44 -07:00
parent 43c93e74e7
commit bc29073ab1
No known key found for this signature in database
GPG key ID: 0DEF1C662C9033F7
2 changed files with 32 additions and 9 deletions

View file

@ -14,6 +14,8 @@ class CustomEmoji extends Model
const SCAN_RE = "/(?<=[^[:alnum:]:]|\n|^):([a-zA-Z0-9_]{2,}):(?=[^[:alnum:]:]|$)/x"; const SCAN_RE = "/(?<=[^[:alnum:]:]|\n|^):([a-zA-Z0-9_]{2,}):(?=[^[:alnum:]:]|$)/x";
const CACHE_KEY = "pf:custom_emoji:"; const CACHE_KEY = "pf:custom_emoji:";
protected $guarded = [];
public static function scan($text, $activitypub = false) public static function scan($text, $activitypub = false)
{ {
if(config('federation.custom_emoji.enabled') == false) { if(config('federation.custom_emoji.enabled') == false) {

View file

@ -6,6 +6,8 @@ use App\Models\CustomEmoji;
use App\Util\ActivityPub\Helpers; use App\Util\ActivityPub\Helpers;
use Illuminate\Support\Facades\Http; use Illuminate\Support\Facades\Http;
use Illuminate\Support\Facades\Cache; use Illuminate\Support\Facades\Cache;
use Illuminate\Support\Facades\Storage;
use Illuminate\Http\Client\RequestException;
class CustomEmojiService class CustomEmojiService
{ {
@ -33,7 +35,13 @@ class CustomEmojiService
return; return;
} }
$res = Http::acceptJson()->get($url); try {
$res = Http::acceptJson()->get($url);
} catch (RequestException $e) {
return;
} catch (\Exception $e) {
return;
}
if($res->successful()) { if($res->successful()) {
$json = $res->json(); $json = $res->json();
@ -57,16 +65,23 @@ class CustomEmojiService
return; return;
} }
$emoji = new CustomEmoji; $emoji = CustomEmoji::firstOrCreate([
$emoji->shortcode = $json['name']; 'shortcode' => $json['name'],
$emoji->uri = $json['id']; 'domain' => parse_url($json['id'], PHP_URL_HOST)
$emoji->domain = parse_url($json['id'], PHP_URL_HOST); ], [
$emoji->image_remote_url = $json['icon']['url']; 'uri' => $json['id'],
$emoji->save(); 'image_remote_url' => $json['icon']['url']
]);
if($emoji->wasRecentlyCreated == false) {
if(Storage::exists('public/' . $emoji->media_path)) {
Storage::delete('public/' . $emoji->media_path);
}
}
$ext = '.' . last(explode('/', $json['icon']['mediaType'])); $ext = '.' . last(explode('/', $json['icon']['mediaType']));
$dest = storage_path('app/public/emoji/') . $emoji->id . $ext; $dest = storage_path('app/public/emoji/') . $emoji->id . $ext;
copy($emoji->image_remote_url, $dest); copy($json['icon']['url'], $dest);
$emoji->media_path = 'emoji/' . $emoji->id . $ext; $emoji->media_path = 'emoji/' . $emoji->id . $ext;
$emoji->save(); $emoji->save();
@ -84,7 +99,13 @@ class CustomEmojiService
public static function headCheck($url) public static function headCheck($url)
{ {
$res = Http::head($url); try {
$res = Http::head($url);
} catch (RequestException $e) {
return false;
} catch (\Exception $e) {
return false;
}
if(!$res->successful()) { if(!$res->successful()) {
return false; return false;