Repository: ckan/ckan

Issue:
RSS Feed uses package_read instead of package_show
**CKAN version**
2.8.6
Seems to be present in master

**Describe the bug**
RSS output generated by feed results in:

"Bad request - Action name not known: package_read"

Due to package_read being used instead of package_show

https://github.com/ckan/ckan/blob/f8d26482f6c0164a8f6b3b3b0b627df529fedef0/ckan/views/feed.py#L177

**Steps to reproduce**
Configure RSS feed for organization or dataset

**Expected behavior**
RSS reader should direct user to organization/dataset

**Additional details**
Sample XML

```
<?xml version="1.0" encoding="utf-8"?>
<feed xmlns="http://www.w3.org/2005/Atom" xml:lang="en"><title>CKAN - Custom query</title><link href="/dataset?query=q&amp;query=name%3Araw-leaf-tobacco-registrant-list" rel="alternate"></link><link href="/feeds/custom.atom?query=q&amp;query=name%3Araw-leaf-tobacco-registrant-list" rel="self"></link><id>http://localhost/feeds/custom.atom</id><updated>2021-04-09T18:04:21Z</updated><author><name>default</name></author><subtitle>Recently created or updated datasets on CKAN. Custom query: 'name:raw-leaf-tobacco-registrant-list'</subtitle><link href="/feeds/custom.atom?query=q&amp;query=name%3Araw-leaf-tobacco-registrant-list" rel="first"></link><link href="/feeds/custom.atom?query=page&amp;query=1" rel="last"></link><entry><title>Raw leaf tobacco registrant list</title><link href="http://localhost/api/3/action/package_read?id=12a41b0f-3e51-4652-9756-962909b7fd78" rel="alternate"></link><author><name></name><email></email></author><id>http://localhost/dataset/12a41b0f-3e51-4652-9756-962909b7fd78</id><summary type="html">This list allows you to identify legal entities authorized by the Minister of
Finance, under the [Tobacco Tax Act](https://www.ontario.ca/laws/statute/90t10), to process, sell,
distribute, import, export and transport raw leaf tobacco products.

Raw leaf tobacco includes all varieties of unmanufactured tobacco grown in or
brought into Ontario, including flueâ€‘cured, darkâ€‘fireâ€‘cured/darkâ€‘airâ€‘cured
(also known as black) and burley tobacco.

Registrants on this list hold a valid registration certificate, permit or
designation.

This data includes:

  * name and address of legal entity
  * whether the legal entity is a processor, dealer, importer/exporter of raw leaf tobacco
  * whether the legal entity has an Interjurisdictional Transporter Registration Certificate
  * the date any changes were made to the legal entity's registrant data, if applicable.

</summary><link length="15702" href="http://localhost/api/3/action/package_show?id=raw-leaf-tobacco-registrant-list" type="application/json" rel="enclosure"></link><category term="Economy and Business"></category><category term="Gouvernement et finances"></category><category term="Government and Finance"></category><category term="ImpÃ´ts et avantages fiscaux"></category><category term="Taxes and benefits"></category><category term="Ã‰conomie et affaires"></category><updated>2021-04-09T18:04:21Z</updated><published>2021-03-09T18:58:47Z</published></entry></feed>
```



Discussion:

Resolving patch:
diff --git a/ckan/views/feed.py b/ckan/views/feed.py
--- a/ckan/views/feed.py
+++ b/ckan/views/feed.py
@@ -174,7 +174,7 @@ def output_feed(results, feed_title, feed_description, feed_link, feed_url,
             title=pkg.get(u'title', u''),
             link=h.url_for(
                 u'api.action',
-                logic_function=u'package_read',
+                logic_function=u'package_show',
                 id=pkg['id'],
                 ver=3,
                 _external=True),


Test patch: