fix : add feed icon to rss2.xml

atom.xml

@@ -1,7 +1,13 @@
 <?xml version="1.0" encoding="utf-8"?>
 <feed xmlns="https://www.w3.org/2005/Atom">
   <title>{{ config.title }}</title>
-  {% if icon %}<icon>{{ icon }}</icon>{% endif %}
+  {% if icon %}
+  <image>
+    <url>{{ icon }}</url>
+    <title>{{ config.title }}</title>
+    <link>{{ url }}</link>
+  </image>
+  {% endif %}
   {% if config.subtitle %}<subtitle>{{ config.subtitle }}</subtitle>{% endif %}
   <link href="{{ feed_url | uriencode }}" rel="self"/>
   {% if config.feed.hub %}<link href="{{ config.feed.hub | uriencode }}" rel="hub"/>{% endif %}

lib/generator.js

@@ -4,7 +4,7 @@ const nunjucks = require('nunjucks');
 const env = new nunjucks.Environment();
 const { join } = require('path');
 const { readFileSync } = require('fs');
-const { encodeURL, gravatar } = require('hexo-util');
+const { encodeURL, gravatar, full_url_for } = require('hexo-util');
 
 env.addFilter('uriencode', str => {
   return encodeURL(str);
@@ -35,14 +35,14 @@ module.exports = function(locals) {
   if (url[url.length - 1] !== '/') url += '/';
 
   let icon = '';
-  if (feedConfig.icon) icon = url + encodeURI(feedConfig.icon);
+  if (feedConfig.icon) icon = full_url_for.call(this, feedConfig.icon);
   else if (config.email) icon = gravatar(config.email);
 
   const xml = template.render({
-    config: config,
-    url: url,
-    icon: icon,
-    posts: posts,
+    config,
+    url,
+    icon,
+    posts,
     feed_url: config.root + feedConfig.path
   });
 

rss2.xml

@@ -5,6 +5,13 @@
   <channel>
     <title>{{ config.title }}</title>
     <link>{{ url }}</link>
+    {% if icon %}
+    <image>
+      <url>{{ icon }}</url>
+      <title>{{ config.title }}</title>
+      <link>{{ url }}</link>
+    </image>
+    {% endif %}
     <atom:link href="{{ feed_url | uriencode }}" rel="self" type="application/rss+xml"/>
     {% if config.feed.hub %}<atom:link href="{{ config.feed.hub | uriencode }}" rel="hub"/>{% endif %}
     <description>{{ config.description }}</description>

test/index.js

@@ -7,7 +7,7 @@ const env = new nunjucks.Environment();
 const { join } = require('path');
 const { readFileSync } = require('fs');
 const cheerio = require('cheerio');
-const { encodeURL } = require('hexo-util');
+const { encodeURL, full_url_for } = require('hexo-util');
 
 env.addFilter('uriencode', str => {
   return encodeURI(str);
@@ -235,6 +235,35 @@ describe('Feed generator', () => {
     };
     checkURL('http://localhost/', '/', 'item:nth-of-type(3)>enclosure');
   });
+
+  it('Icon (atom)', () => {
+    hexo.config.url = 'http://example.com';
+    hexo.config.root = '/';
+
+    hexo.config.feed = {
+      type: 'atom',
+      path: 'atom.xml',
+      icon: 'icon.svg'
+    };
+
+    const result = generator(locals);
+    const $ = cheerio.load(result.data);
+
+    $('feed>icon').text().should.eql(full_url_for.call(hexo, hexo.config.feed.icon));
+  });
+
+  it('Icon (atom) - no icon', () => {
+    hexo.config.feed = {
+      type: 'atom',
+      path: 'atom.xml',
+      icon: undefined
+    };
+
+    const result = generator(locals);
+    const $ = cheerio.load(result.data);
+
+    $('feed>icon').length.should.eql(0);
+  });
 });
 
 describe('Autodiscovery', () => {