search

asvae / laravel-api-tester

Test your routes without hassle
http://laravel-api-tester.asva.by/
MIT License
354 stars 52 forks source link

Just pulled it in and got an issue registering its routes #30

Closed gocanto closed 8 years ago

gocanto commented 8 years ago

[LogicException] Unable to prepare route [api-tester/* routes should not be cached] for serialization. Uses Closure.

asvae commented 8 years ago

@gocanto This is an intended feature : 3. Caching routes while using api-tester is dangerous.

gocanto commented 8 years ago

how so?

asvae commented 8 years ago

@gocanto Well, my point is you cache routes only in production enviroment. Api tester is not intended for production usage as it provides no security. Hence, weird exception.

Any arguments against the feature? Except for that it's weird (which I know). : 3

gocanto commented 8 years ago

Whether you canchebthe rountes or not, this is a pkg that never is gonna be in prod, so there is not reason to have this set up. This is my thought

asvae commented 8 years ago

It could end up in prod by mistake. It's merely fool proof measure.