Dockery/Jump
1
0
Fork 0
mirror of https://github.com/daledavies/jump.git synced 2026-05-06 19:26:59 +02:00
Code Issues Packages Projects Releases Activity

Fix Issue #82: ampersand in url in sites.json breaks icon

Browse Source
This commit is contained in:
Dale Davies
2023-04-04 14:15:27 +01:00
parent 8aad20147b
commit 260e886db8
3 changed files with 7 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

9
jumpapp/classes/API/Icon.php
View File

@@ -18,14 +18,15 @@ use \Jump\Exceptions\APIException;
class Icon extends AbstractAPI {
public function get_output(): string {
if (!isset($this->routeparams['siteurl']) || empty($this->routeparams['siteurl'])) {
throw new APIException('The siteurl query parameter is not provided or empty');
if (!isset($this->routeparams['siteid']) || empty($this->routeparams['siteid'])) {
throw new APIException('The siteid query parameter is not provided or empty');
}
$sites = new \Jump\Sites($this->config, $this->cache);
$siteurl = filter_var($this->routeparams['siteurl'], FILTER_SANITIZE_URL);
$site = $sites->get_site_by_url($siteurl);
# A site ID can contain lowercase a-z, 0-9 and the "-" (dash) character only.
$siteid = preg_replace("/[^a-z0-9-]/", "", $this->routeparams['siteid']);
$site = $sites->get_site_by_id($siteid);
$imagedata = $site->get_favicon_image_data();