Nightly build

7 views
Skip to first unread message

br...@bradwood.com

unread,
Dec 17, 2010, 12:27:58 PM12/17/10
to ColdBox List
A couple notes about the nightly build I downloaded from coldbox.org last night.  I know it's a nightly, but I figured I'd mention it just so you knew.   I tried to figure out the github repo, but I got too fed up with the quirky git clients that I couldn't get working.  I'd love some help with a good git client. 

  1. The SES interceptor was throwing errors about invalid extensions.  Most of my SES URLs end with random values like ehUser/view/userID/12345 and "12345" isn't meant to be an extension, let alone a valid one.  I fixed by adding <cfset setExtensionDetection( false )>  to my routes.cfm file.  Is extension detection going to always default to "true"?
  2. The cookie storage plugin was updated to always JSON everything as opposed to simply storing simple values and wddxing everything else.  I have no problem with that change, but it freaked out with my existing cookie values since they weren't JSON.  I don't see how I could cut over to such a change in production without clearing all my users' cookies.  I think the get() method in the cookie plugin might need to do some isWDDX and isJSON checks and not assume everything it is pulling out is JSON for the next release.
Thanks!

~Brad

Luis Majano

unread,
Dec 17, 2010, 12:51:48 PM12/17/10
to col...@googlegroups.com

A couple notes about the nightly build I downloaded from coldbox.org last night.  I know it's a nightly, but I figured I'd mention it just so you knew.   I tried to figure out the github repo, but I got too fed up with the quirky git clients that I couldn't get working.  I'd love some help with a good git client. 

Brad are you on windows or Mac? Anyways, I use SmartGIT, by far the best git client I have used and it keeps getting better and better, available for both windows and mac.

 
  1. The SES interceptor was throwing errors about invalid extensions.  Most of my SES URLs end with random values like ehUser/view/userID/12345 and "12345" isn't meant to be an extension, let alone a valid one.  I fixed by adding <cfset setExtensionDetection( false )>  to my routes.cfm file.  Is extension detection going to always default to "true"?
It is by default turned on, but it should not give errors like that, maybe we need to test it more.  But all my apps work ok using the latest from github, so this might be something else.
 
  1. The cookie storage plugin was updated to always JSON everything as opposed to simply storing simple values and wddxing everything else.  I have no problem with that change, but it freaked out with my existing cookie values since they weren't JSON.  I don't see how I could cut over to such a change in production without clearing all my users' cookies.  I think the get() method in the cookie plugin might need to do some isWDDX and isJSON checks and not assume everything it is pulling out is JSON for the next release.
Well, should we have some compatibility code in it where if it is wddx then it should clear it out? So it starts fresh again?
 
Thanks!

~Brad

--
You received this message because you are subscribed to the Google Groups "ColdBox Platform" group.
To post to this group, send email to col...@googlegroups.com
To unsubscribe from this group, send email to coldbox-u...@googlegroups.com
For more options, visit this group at http://groups-beta.google.com/group/coldbox
For News, visit http://blog.coldbox.org
For Documentation, visit http://wiki.coldbox.org

Nolan Dubeau

unread,
Dec 17, 2010, 1:00:38 PM12/17/10
to col...@googlegroups.com
For Mac, Tower is also a nice GIT client - http://www.git-tower.com/

Nolan

br...@bradwood.com

unread,
Dec 17, 2010, 1:40:33 PM12/17/10
to col...@googlegroups.com
> Brad are you on windows or Mac? Anyways, I use SmartGIT, by far the best git client I have used and it keeps getting better and better, available for both windows and mac.

Windoze.  SmartGit is what I downloaded but I was having troubles figuring it out.  It kept complaining that I needed to point it to a GIT executable and I tried installing it (though I was really just randomly trying download links because I was fairly clueless about what I needed to download) but it was stuck at that message no matter what I did.  I was hoping for something nice and easy like a single download installer w/everything I needed that would just work, but it wasn't worth 45 minutes of my time last night just so I could get the silly code downloaded.  :)  All I wanted was a client to connect to a repo, but it seemed intent on having me install the whole shebang.  Since I already have subversion installed for my work projects, I wasn't too keen on littering my system with desktop, start menu, and quick launch shortcuts; not to mention windows explorer integrations just so I could get a ColdBox nightly.  As long as the nightly build from the main ColdBox site works, I think I'll be happy just to use it on the few occasions that I need to for now.

> It is by default turned on, but it should not give errors like that, maybe we need to test it more.  But all my apps work ok using the latest from github, so this might be something else.

The error seems to happen when you are using packages for your handlers.  Here is an example of a URL that is erroring for me:


findRoute works find the first time since it uses the following requestString "/contactManager/ehAuthentication/showLogin ".
But when it recurses to resolve the package, it passes in "contactManager.ehAuthentication/showLogin/" which has a period in it. 
It thinks that "ehAuthentication/showLogin" is the extension and throws an error on line 468.
 

> Well, should we have some compatibility code in it where if it is wddx then it should clear it out? So it starts fresh again?
 
I thought about that.  The drawback there, is that your application will lose the benefit of any previously saved values in the CookieStorage mechanism after upgrade.  For my users, it means my site will no longer remember their login like they asked, and will have reset all their preferences as to what accordions are expanded for them when they log in to my site.  I would prefer to retain the data my users have in their cookies and not trash it if you know what I mean. 

My recommendation for line 133 of the cookie storage plugin is to maintain backwards compatibility by checking isJSON(), then isWDDX() (and de-serializing appropriately), and the "else" just returning a simple value.  That will make the transition seamless.

Thanks!

~Brad

Luis Majano

unread,
Dec 17, 2010, 9:42:12 PM12/17/10
to col...@googlegroups.com
Ok Brad,

I reworked the extension detection as it had to be done once per routing instead of the recursion possibilities.  I updated it and the tests, so it should be golden now.

As for git for windows I do this:

2) Install SmartGit, use it.

That's it.

As for the wddx, I will try to add a conversion process to it.

Luis F. Majano
President
Ortus Solutions, Corp

ColdBox Platform: http://www.coldbox.org
Linked In: http://www.linkedin.com/pub/3/731/483
Blog: http://www.luismajano.com
IECFUG Manager: http://www.iecfug.com


--

Luis Majano

unread,
Dec 17, 2010, 10:10:34 PM12/17/10
to col...@googlegroups.com
Ok cookiestorage compat mode is now online!!

Luis F. Majano
President
Ortus Solutions, Corp

ColdBox Platform: http://www.coldbox.org
Linked In: http://www.linkedin.com/pub/3/731/483
Blog: http://www.luismajano.com
IECFUG Manager: http://www.iecfug.com


Curt Gratz

unread,
Dec 17, 2010, 10:16:48 PM12/17/10
to col...@googlegroups.com
If your on Windows I use TortoiseGit. I used TortoiseSVN before so it was an easy switch for me.

http://code.google.com/p/tortoisegit/

Curt Gratz
Computer Know How

________________________________

From: col...@googlegroups.com on behalf of Nolan Dubeau
Sent: Fri 12/17/2010 12:00 PM
To: col...@googlegroups.com
Subject: Re: [coldbox:7286] Nightly build


For Mac, Tower is also a nice GIT client - http://www.git-tower.com/

Nolan

On 2010-12-17, at 12:51 PM, Luis Majano wrote:


A couple notes about the nightly build I downloaded from coldbox.org <http://coldbox.org/> last night. I know it's a nightly, but I figured I'd mention it just so you knew. I tried to figure out the github repo, but I got too fed up with the quirky git clients that I couldn't get working. I'd love some help with a good git client.


Brad are you on windows or Mac? Anyways, I use SmartGIT, by far the best git client I have used and it keeps getting better and better, available for both windows and mac.

1. The SES interceptor was throwing errors about invalid extensions. Most of my SES URLs end with random values like ehUser/view/userID/12345 and "12345" isn't meant to be an extension, let alone a valid one. I fixed by adding <cfset setExtensionDetection( false )> to my routes.cfm file. Is extension detection going to always default to "true"?

It is by default turned on, but it should not give errors like that, maybe we need to test it more. But all my apps work ok using the latest from github, so this might be something else.

1. The cookie storage plugin was updated to always JSON everything as opposed to simply storing simple values and wddxing everything else. I have no problem with that change, but it freaked out with my existing cookie values since they weren't JSON. I don't see how I could cut over to such a change in production without clearing all my users' cookies. I think the get() method in the cookie plugin might need to do some isWDDX and isJSON checks and not assume everything it is pulling out is JSON for the next release.

Well, should we have some compatibility code in it where if it is wddx then it should clear it out? So it starts fresh again?


Thanks!

~Brad

--
You received this message because you are subscribed to the Google Groups "ColdBox Platform" group.
To post to this group, send email to col...@googlegroups.com
To unsubscribe from this group, send email to coldbox-u...@googlegroups.com
For more options, visit this group at http://groups-beta.google.com/group/coldbox

For News, visit http://blog.coldbox.org <http://blog.coldbox.org/>
For Documentation, visit http://wiki.coldbox.org <http://wiki.coldbox.org/>

--
You received this message because you are subscribed to the Google Groups "ColdBox Platform" group.
To post to this group, send email to col...@googlegroups.com
To unsubscribe from this group, send email to coldbox-u...@googlegroups.com
For more options, visit this group at http://groups-beta.google.com/group/coldbox

For News, visit http://blog.coldbox.org <http://blog.coldbox.org/>
For Documentation, visit http://wiki.coldbox.org <http://wiki.coldbox.org/>


--
You received this message because you are subscribed to the Google Groups "ColdBox Platform" group.
To post to this group, send email to col...@googlegroups.com
To unsubscribe from this group, send email to coldbox-u...@googlegroups.com
For more options, visit this group at http://groups-beta.google.com/group/coldbox

For News, visit http://blog.coldbox.org <http://blog.coldbox.org/>
For Documentation, visit http://wiki.coldbox.org <http://wiki.coldbox.org/>

winmail.dat

Brad Wood

unread,
Dec 18, 2010, 1:49:18 AM12/18/10
to col...@googlegroups.com
I reworked the extension detection as it had to be done once per routing instead of the recursion possibilities.  I updated it and the tests, so it should be golden now.
Sweet!  Thanks!
 
As for git for windows I do this:
 
Ok, I might give it another try.  I was just in a hurry the other day and my patience was running thin when I couldn't get it to work.   :)
 
Thanks!
 
~Brad

Brad Wood

unread,
Dec 18, 2010, 1:51:13 AM12/18/10
to col...@googlegroups.com
What's that?  It's the sound of my users thanking you!
 
All three of them.
 
:)
 
~Brad

Brad Wood

unread,
Dec 18, 2010, 1:56:36 AM12/18/10
to col...@googlegroups.com
That sounds promising.

Is it an explorer extension like TortoiseSVN? Does it also require the Git
binary to be installed.

SmartGit (can't help but want to call it "Git Smart", hee hee) was kind of
freaking me out with all its buttons and menus when all I wanted was to fill
a single folder on my hard drive with ColdBox nightly goodness from the
intertubes.

I'll take a look at TortoiseGit. :)

~Brad

----- Original Message -----
From: "Curt Gratz" <gra...@compknowhow.com>
To: <col...@googlegroups.com>

John Whish

unread,
Dec 18, 2010, 2:01:03 PM12/18/10
to col...@googlegroups.com

I've been using eGit which works fine with ColdFusion Builder and CFEclipse.

http://www.eclipse.org/egit/download/

--
You received this message because you are subscribed to the Google Groups "ColdBox Platform" group.
To post to this group, send email to col...@googlegroups.com
To unsubscribe from this group, send email to coldbox-u...@googlegroups.com
For more options, visit this group at http://groups-beta.google.com/group/coldbox
For News, visit http://blog.coldbox.org
Reply all
Reply to author
Forward
0 new messages