Issue #3258247 by Gábor Hojtsy, chrisfromredfin, gaurav.kapoor, hmendes: Add...

Issue #3258247 by Gábor Hojtsy, chrisfromredfin, gaurav.kapoor, hmendes: Add permissions to the project browser route

project_browser.routing.yml

@@ -5,10 +5,7 @@ project_browser.api_project_get_all:
     _controller: '\Drupal\project_browser\Controller\DrupalOrgProxyController::getAllProjects'
     _title: 'Get all projects'
   requirements:
-    #_permission: 'manage own subscriptions+administer subscriptions'
-    #_user_is_logged_in: 'TRUE'
-    # @todo Undo this access bypass!
-    _access: 'TRUE'
+    _permission: 'administer modules'
   #options:
   #  no_cache: 'TRUE'
 project_browser.api_project_get_releases:
@@ -18,10 +15,7 @@ project_browser.api_project_get_releases:
     _controller: '\Drupal\project_browser\Controller\DrupalOrgProxyController::getProjectReleases'
     _title: 'Get all project releases'
   requirements:
-    #_permission: 'manage own subscriptions+administer subscriptions'
-    #_user_is_logged_in: 'TRUE'
-    # @todo Undo this access bypass!
-    _access: 'TRUE'
+    _permission: 'administer modules'
   #options:
   #  no_cache: 'TRUE'
 project_browser.browse:
@@ -30,7 +24,4 @@ project_browser.browse:
     _controller: '\Drupal\project_browser\Controller\BrowserController::browse'
     _title: 'Browse Drupal.org projects'
   requirements:
-    #_permission: 'manage own subscriptions+administer subscriptions'
-    #_user_is_logged_in: 'TRUE'
-    # @todo Undo this access bypass!
-    _access: 'TRUE'
+    _permission: 'administer modules'

tests/src/FunctionalJavascript/ProjectBrowserUiTest.php

@@ -30,7 +30,7 @@ class ProjectBrowserUiTest extends WebDriverTestBase {
     parent::setUp();
 
     $this->drupalLogin($this->drupalCreateUser([
-      // @todo Update once project_browser.browse has requirements.
+      'administer modules'
     ]));
   }