Restrict fluent route attributes to known attributes

src/Illuminate/Routing/RouteRegistrar.php

@@ -3,6 +3,7 @@
 namespace Illuminate\Routing;
 
 use Closure;
+use BadMethodCallException;
 
 class RouteRegistrar
 {
@@ -29,6 +30,15 @@ class RouteRegistrar
         'get', 'post', 'put', 'patch', 'delete', 'options', 'any',
     ];
 
+    /**
+     * The attributes that can be set through this class.
+     *
+     * @var array
+     */
+    protected $allowedAttributes = [
+        'as', 'domain', 'middleware', 'name', 'namespace', 'prefix',
+    ];
+
     /**
      * The attributes that are aliased.
      *
@@ -149,6 +159,10 @@ public function __call($method, $parameters)
             return $this->registerRoute($method, ...$parameters);
         }
 
-        return $this->attribute($method, $parameters[0]);
+        if (in_array($method, $this->allowedAttributes)) {
+            return $this->attribute($method, $parameters[0]);
+        }
+
+        throw new BadMethodCallException("Method [{$method}] does not exist.");
     }
 }

tests/Routing/RouteRegistrarTest.php

@@ -155,6 +155,26 @@ public function testCanRegisterGroupWithDomain()
         $this->assertEquals('{account}.myapp.com', $this->getRoute()->domain());
     }
 
+    public function testCanRegisterGroupWithDomainAndNamePrefix()
+    {
+        $this->router->domain('{account}.myapp.com')->name('api.')->group(function ($router) {
+            $router->get('users', 'UsersController@index')->name('users');
+        });
+
+        $this->assertEquals('{account}.myapp.com', $this->getRoute()->domain());
+        $this->assertEquals('api.users', $this->getRoute()->getName());
+    }
+
+    /**
+     * @expectedException \BadMethodCallException
+     */
+    public function testRegisteringNonApprovedAttributesThrows()
+    {
+        $this->router->domain('foo')->missing('bar')->group(function ($router) {
+            //
+        });
+    }
+
     public function testCanRegisterResource()
     {
         $this->router->middleware('resource-middleware')