[Lift] Re: lift-openid broken

2009-08-18 Thread Vassil 

 Done.

Great, propagated to maven repo now. Thanks.

--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
Lift group.
To post to this group, send email to liftweb@googlegroups.com
To unsubscribe from this group, send email to 
liftweb+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/liftweb?hl=en
-~--~~~~--~~--~--~---

[Lift] Re: lift-openid broken

2009-08-17 Thread marius d. 

Thank you for pointing this out. Most likely I'll make the corrections
today.

Br's,
Marius

On Aug 17, 1:48 am, Vassil vdic...@gmail.com wrote:
 Hello guys,

 It seems the HTTP refactoring around 10 days ago broke OpenId. Here's
 a small patch that made it work for me:

 --- dpp-liftweb-e845b3c129baf16bc35b46df9ee5b5049d4aee3a/lift-openid/
 src/main/scala/net/liftweb/openid/OpenId.scala     2009-08-16
 17:36:58.0 +0300
 +++ dpp-liftweb-fixed/lift-openid/src/main/scala/net/liftweb/openid/
 OpenId.scala        2009-08-17 01:43:46.0 +0300
 @@ -267,7 +267,7 @@

      // retrieve the previously stored discovery information
      val discovered = httpReq.session.attribute(openid-disc) match {
 -      case Full(d: DiscoveryInformation)= d
 +      case d: DiscoveryInformation= d
        case _ = throw ResponseShortcutException.redirect(/)
      }

 @@ -275,7 +275,7 @@
      var receivingURL = httpReq.url
      val queryString = httpReq.queryString openOr 
      if (queryString != null  queryString.length()  0) {
 -      receivingURL += ? + httpReq.queryString;
 +      receivingURL += ? + queryString;
      }
--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
Lift group.
To post to this group, send email to liftweb@googlegroups.com
To unsubscribe from this group, send email to 
liftweb+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/liftweb?hl=en
-~--~~~~--~~--~--~---

[Lift] Re: lift-openid broken

2009-08-17 Thread David Pollak 
On Sun, Aug 16, 2009 at 11:52 PM, marius d. marius.dan...@gmail.com wrote:


 Thank you for pointing this out. Most likely I'll make the corrections
 today.


Please let us know when the changes are rolled in.




 Br's,
 Marius

 On Aug 17, 1:48 am, Vassil vdic...@gmail.com wrote:
  Hello guys,
 
  It seems the HTTP refactoring around 10 days ago broke OpenId. Here's
  a small patch that made it work for me:
 
  --- dpp-liftweb-e845b3c129baf16bc35b46df9ee5b5049d4aee3a/lift-openid/
  src/main/scala/net/liftweb/openid/OpenId.scala 2009-08-16
  17:36:58.0 +0300
  +++ dpp-liftweb-fixed/lift-openid/src/main/scala/net/liftweb/openid/
  OpenId.scala2009-08-17 01:43:46.0 +0300
  @@ -267,7 +267,7 @@
 
   // retrieve the previously stored discovery information
   val discovered = httpReq.session.attribute(openid-disc) match {
  -  case Full(d: DiscoveryInformation)= d
  +  case d: DiscoveryInformation= d
 case _ = throw ResponseShortcutException.redirect(/)
   }
 
  @@ -275,7 +275,7 @@
   var receivingURL = httpReq.url
   val queryString = httpReq.queryString openOr 
   if (queryString != null  queryString.length()  0) {
  -  receivingURL += ? + httpReq.queryString;
  +  receivingURL += ? + queryString;
   }
 



-- 
Lift, the simply functional web framework http://liftweb.net
Beginning Scala http://www.apress.com/book/view/1430219890
Follow me: http://twitter.com/dpp
Git some: http://github.com/dpp

--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
Lift group.
To post to this group, send email to liftweb@googlegroups.com
To unsubscribe from this group, send email to 
liftweb+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/liftweb?hl=en
-~--~~~~--~~--~--~---

[Lift] Re: lift-openid broken

2009-08-17 Thread marius d. 

Done.

Br's,
Marius

On Aug 17, 9:39 pm, David Pollak feeder.of.the.be...@gmail.com
wrote:
 On Sun, Aug 16, 2009 at 11:52 PM, marius d. marius.dan...@gmail.com wrote:

  Thank you for pointing this out. Most likely I'll make the corrections
  today.

 Please let us know when the changes are rolled in.





  Br's,
  Marius

  On Aug 17, 1:48 am, Vassil vdic...@gmail.com wrote:
   Hello guys,

   It seems the HTTP refactoring around 10 days ago broke OpenId. Here's
   a small patch that made it work for me:

   --- dpp-liftweb-e845b3c129baf16bc35b46df9ee5b5049d4aee3a/lift-openid/
   src/main/scala/net/liftweb/openid/OpenId.scala     2009-08-16
   17:36:58.0 +0300
   +++ dpp-liftweb-fixed/lift-openid/src/main/scala/net/liftweb/openid/
   OpenId.scala        2009-08-17 01:43:46.0 +0300
   @@ -267,7 +267,7 @@

        // retrieve the previously stored discovery information
        val discovered = httpReq.session.attribute(openid-disc) match {
   -      case Full(d: DiscoveryInformation)= d
   +      case d: DiscoveryInformation= d
          case _ = throw ResponseShortcutException.redirect(/)
        }

   @@ -275,7 +275,7 @@
        var receivingURL = httpReq.url
        val queryString = httpReq.queryString openOr 
        if (queryString != null  queryString.length()  0) {
   -      receivingURL += ? + httpReq.queryString;
   +      receivingURL += ? + queryString;
        }

 --
 Lift, the simply functional web frameworkhttp://liftweb.net
 Beginning Scalahttp://www.apress.com/book/view/1430219890
 Follow me:http://twitter.com/dpp
 Git some:http://github.com/dpp
--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
Lift group.
To post to this group, send email to liftweb@googlegroups.com
To unsubscribe from this group, send email to 
liftweb+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/liftweb?hl=en
-~--~~~~--~~--~--~---

[Lift] Re: lift-openid broken

2009-08-17 Thread David Pollak 
On Mon, Aug 17, 2009 at 12:35 PM, marius d. marius.dan...@gmail.com wrote:


 Done.


Tnx




 Br's,
 Marius

 On Aug 17, 9:39 pm, David Pollak feeder.of.the.be...@gmail.com
 wrote:
  On Sun, Aug 16, 2009 at 11:52 PM, marius d. marius.dan...@gmail.com
 wrote:
 
   Thank you for pointing this out. Most likely I'll make the corrections
   today.
 
  Please let us know when the changes are rolled in.
 
 
 
 
 
   Br's,
   Marius
 
   On Aug 17, 1:48 am, Vassil vdic...@gmail.com wrote:
Hello guys,
 
It seems the HTTP refactoring around 10 days ago broke OpenId. Here's
a small patch that made it work for me:
 
--- dpp-liftweb-e845b3c129baf16bc35b46df9ee5b5049d4aee3a/lift-openid/
src/main/scala/net/liftweb/openid/OpenId.scala 2009-08-16
17:36:58.0 +0300
+++ dpp-liftweb-fixed/lift-openid/src/main/scala/net/liftweb/openid/
OpenId.scala2009-08-17 01:43:46.0 +0300
@@ -267,7 +267,7 @@
 
 // retrieve the previously stored discovery information
 val discovered = httpReq.session.attribute(openid-disc) match
 {
-  case Full(d: DiscoveryInformation)= d
+  case d: DiscoveryInformation= d
   case _ = throw ResponseShortcutException.redirect(/)
 }
 
@@ -275,7 +275,7 @@
 var receivingURL = httpReq.url
 val queryString = httpReq.queryString openOr 
 if (queryString != null  queryString.length()  0) {
-  receivingURL += ? + httpReq.queryString;
+  receivingURL += ? + queryString;
 }
 
  --
  Lift, the simply functional web frameworkhttp://liftweb.net
  Beginning Scalahttp://www.apress.com/book/view/1430219890
  Follow me:http://twitter.com/dpp
  Git some:http://github.com/dpp
 



-- 
Lift, the simply functional web framework http://liftweb.net
Beginning Scala http://www.apress.com/book/view/1430219890
Follow me: http://twitter.com/dpp
Git some: http://github.com/dpp

--~--~-~--~~~---~--~~
You received this message because you are subscribed to the Google Groups 
Lift group.
To post to this group, send email to liftweb@googlegroups.com
To unsubscribe from this group, send email to 
liftweb+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/liftweb?hl=en
-~--~~~~--~~--~--~---