[Jifty-commit] r5135 - in jifty/trunk: lib/Jifty/Plugin lib/Jifty/Plugin/OAuth

[Jifty commits]

Author: sartak
Date: Tue Feb 19 17:18:19 2008
New Revision: 5135

Modified:
   jifty/trunk/   (props changed)
   jifty/trunk/lib/Jifty/Plugin/OAuth.pm
   jifty/trunk/lib/Jifty/Plugin/OAuth/View.pm

Log:
 r51939 at onn:  sartak | 2008-02-19 17:12:40 -0500
 Small OAuth doc improvements


Modified: jifty/trunk/lib/Jifty/Plugin/OAuth.pm
==============================================================================
--- jifty/trunk/lib/Jifty/Plugin/OAuth.pm	(original)
+++ jifty/trunk/lib/Jifty/Plugin/OAuth.pm	Tue Feb 19 17:18:19 2008
@@ -20,7 +20,7 @@
 
 =head1 USAGE
 
-Add the following to your site_config.yml
+Add the following to your config:
 
  framework:
    Plugins:
@@ -66,9 +66,12 @@
 You must not allow public access to C</oauth/authorize>. C</oauth/authorize>
 depends on having the user be logged in.
 
+You should allow public access to C</oauth>. This has some information for
+consumers.
+
 There is currently no way for consumers to add themselves. This might change in
-the future, but it would be a nondefault configuration. Consumers must
-contact you and provide you with the following data:
+the future, with an OAuth extension. Consumers must contact you and provide you
+with the following data:
 
 =over 4
 

Modified: jifty/trunk/lib/Jifty/Plugin/OAuth/View.pm
==============================================================================
--- jifty/trunk/lib/Jifty/Plugin/OAuth/View.pm	(original)
+++ jifty/trunk/lib/Jifty/Plugin/OAuth/View.pm	Tue Feb 19 17:18:19 2008
@@ -39,7 +39,7 @@
 template 'oauth' => page {
     p {
         b { a { attr { href => "http://oauth.net/" } "OAuth" } };
-        outs " is an open protocol to allow secure authentication to users' private data."
+        outs " is an open protocol to allow secure authentication to users' private data. It's far more secure than users giving out their passwords."
     }
 
     p {
@@ -51,18 +51,14 @@
     }
 
     dl {
-        dt {
-            outs "Request a Request Token";
-            dd { Jifty->web->url(path => '/oauth/request_token') }
-        }
-        dt {
-            outs "Obtain user authorization for a Request Token";
-            dd { Jifty->web->url(path => '/oauth/authorize') }
-        }
-        dt {
-            outs "Exchange a Request Token for an Access Token";
-            dd { Jifty->web->url(path => '/oauth/access_token') }
-        }
+        dt { "Request a Request Token" }
+        dd { Jifty->web->url(path => '/oauth/request_token') }
+
+        dt { "Obtain user authorization for a Request Token" }
+        dd { Jifty->web->url(path => '/oauth/authorize') }
+
+        dt { "Exchange a Request Token for an Access Token" }
+        dd { Jifty->web->url(path => '/oauth/access_token') }
     }
 
     p {
@@ -179,11 +175,19 @@
     div {
         p {
             show '/oauth/consumer';
-            outs ' is trying to access some of your data on this site. If you trust this application, you may grant it access. Note that access is read-only and will expire in one hour.';
+            outs ' is trying to access your data on this site. If you trust this application, you may grant it access. Note that ';
+            strong { "access is unrestricted" };
+            outs ' and will expire in one hour after you click "Allow".';
         }
         p {
             "If you're at all uncomfortable with the idea of someone rifling through your things, click Deny."
         }
+        p {
+            hyperlink(
+                label => "Learn more about OAuth.",
+                url   => "http://oauth.net/",
+            )
+        }
     }
 };