[Jifty-commit] r6666 - in jifty/trunk: lib/Jifty/Plugin/REST

Jifty commits jifty-commit at lists.jifty.org
Wed Mar 18 16:12:44 EDT 2009


Author: sartak
Date: Wed Mar 18 16:12:44 2009
New Revision: 6666

Modified:
   jifty/trunk/   (props changed)
   jifty/trunk/lib/Jifty/Plugin/REST/Dispatcher.pm

Log:
 r81396 at onn:  sartak | 2009-03-18 16:12:12 -0400
 Better explanation for why we don't check is_allowed


Modified: jifty/trunk/lib/Jifty/Plugin/REST/Dispatcher.pm
==============================================================================
--- jifty/trunk/lib/Jifty/Plugin/REST/Dispatcher.pm	(original)
+++ jifty/trunk/lib/Jifty/Plugin/REST/Dispatcher.pm	Wed Mar 18 16:12:44 2009
@@ -428,8 +428,8 @@
         name          => $_[0],
         base          => 'Jifty::Action',
         possibilities => [Jifty->api->visible_actions],
-# XXX: This does not quite work for some applications yet. Older versions of
-# the REST API did not have this explicit restriction.
+# We do not do this check because we want users to see actions on GET requests,
+# like when they're exploring the REST API in their browser.
 #        is_allowed    => sub { Jifty->api->is_allowed(shift) },
     );
 }


More information about the Jifty-commit mailing list