handle /dataset and /dataset/

without it,  /dataset returns 500 instead of 403.

Author: FuhuXia

ckanext/datagov_inventory/plugin.py

@@ -118,9 +118,15 @@ def get_blueprint(self):
 def redirect_homepage():
     if current_user.is_authenticated or g.user:
         CKAN_SITE_URL = config.get("ckan.site_url")
-        return redirect(CKAN_SITE_URL + '/dataset', code=302)
+        return redirect(CKAN_SITE_URL + '/dataset/', code=302)
     else:
         return base.render(u'error/anonymous.html')
 
 
 pusher.add_url_rule('/', view_func=redirect_homepage)
+
+@pusher.before_app_request
+def check_dataset_access():
+    if toolkit.request.path in ('/dataset/', '/dataset'):
+        if not current_user.is_authenticated and not g.user:
+            return base.render(u'error/anonymous.html'), 403

e2e/cypress/integration/public_access.cy.js

@@ -26,6 +26,13 @@ describe('Public Access', () => {
         }).then((response) => {
             expect(response.status).to.eq(403);
         });
+
+        cy.request({
+            url: '/dataset/',
+            failOnStatusCode: false,
+        }).then((response) => {
+            expect(response.status).to.eq(403);
+        });
     });
 
     it('Cannot access the dataset pages', () => {