Browse Source

Hide feed credentials when adding a new feed (#3099)

Avoid showing HTTP Basic Auth credentials in feed error log and in feed
title (when there is no title provided)
Alexandre Alapetite 5 years ago
parent
commit
3f9518b377
1 changed files with 2 additions and 2 deletions
  1. 2 2
      app/Models/Feed.php

+ 2 - 2
app/Models/Feed.php

@@ -287,7 +287,7 @@ class FreshRSS_Feed extends Minz_Model {
 				if ((!$mtime) || $simplePie->error()) {
 					$errorMessage = $simplePie->error();
 					throw new FreshRSS_Feed_Exception(
-						($errorMessage == '' ? 'Unknown error for feed' : $errorMessage) . ' [' . $url . ']'
+						($errorMessage == '' ? 'Unknown error for feed' : $errorMessage) . ' [' . $this->url . ']'
 					);
 				}
 
@@ -302,7 +302,7 @@ class FreshRSS_Feed extends Minz_Model {
 
 					//HTML to HTML-PRE	//ENT_COMPAT except '&'
 					$title = strtr(html_only_entity_decode($simplePie->get_title()), array('<' => '&lt;', '>' => '&gt;', '"' => '&quot;'));
-					$this->_name($title == '' ? $url : $title);
+					$this->_name($title == '' ? $this->url : $title);
 
 					$this->_website(html_only_entity_decode($simplePie->get_link()));
 					$this->_description(html_only_entity_decode($simplePie->get_description()));