aboutsummaryrefslogtreecommitdiff
path: root/app/Controllers/feedController.php
diff options
context:
space:
mode:
authorGravatar Alexandre Alapetite <alexandre@alapetite.fr> 2016-10-20 01:19:59 +0200
committerGravatar Alexandre Alapetite <alexandre@alapetite.fr> 2016-10-20 01:19:59 +0200
commit1893fc61e0e576519f878267fd877247445d1055 (patch)
treeb94ddc05bcf4d0198fca5bf973018641a8282394 /app/Controllers/feedController.php
parent8a45743b9036078344ea362b183358f8ca8e4dd3 (diff)
guid and urls should not contain low/high characters
It looks like SimplePie does not always filter everything Having a character not in latin1 would create MySQL collate errors
Diffstat (limited to 'app/Controllers/feedController.php')
-rwxr-xr-xapp/Controllers/feedController.php4
1 files changed, 3 insertions, 1 deletions
diff --git a/app/Controllers/feedController.php b/app/Controllers/feedController.php
index 8751d2fff..d4a6c9955 100755
--- a/app/Controllers/feedController.php
+++ b/app/Controllers/feedController.php
@@ -314,7 +314,9 @@ class FreshRSS_feed_Controller extends Minz_ActionController {
if (count($entries) > 0) {
$newGuids = array();
foreach ($entries as $entry) {
- $newGuids[] = $entry->guid();
+ $guid = $entry->guid();
+ $guid = filter_var($guid, FILTER_DEFAULT, FILTER_FLAG_STRIP_LOW | FILTER_FLAG_STRIP_HIGH);
+ $newGuids[] = $guid;
}
// For this feed, check existing GUIDs already in database.
$existingHashForGuids = $entryDAO->listHashForFeedGuids($feed->id(), $newGuids);