cachify injects stuff into the global namespace

[lloyd]

$ node
> require('connect-cachify');
{ setup: [Function],
  cachify_js: [Function],
  cachify_css: [Function],
  cachify: [Function] }
> cachify_js
[Function]

In BrowserID, we actually rely on this behavior (maybe accidentally?). In our templates we use cachify functions and we never actually pass them in.

I'd think it might be safer to not inject into the global namespace. I'd write a patch if you agreed, but what do you think?

[seanmonstar]

Keep the global clean! And, I imagine express lets us define default function helpers to templates.

[lloyd]

yes! @zcarter taught me this one:

res.local('cachify_js', cachify.cachify_js);

is all we would have to add to keep things going in browserid.

[ozten]

My bad. Please fix.