On Wed, Jun 25, 2008 at 10:16 AM, Chris Chabot <[EMAIL PROTECTED]> wrote:
> Hey Rajdeep, this looks great! > > Only a few small bits of feedback that aren't really important but i'd > still like to mention: > > # FilesServlet > * Loads the HTML, JS, CSS files from the paths specified -- This servlet > is unique to PHP Shindig Implementation. > > Actually this isn't from the 'path specified' but hard wires /gadgets/files > to <shindig>/javascript. Java land also does this but then through jetty > configuration. The purpose of it allow the examples > (/gadgets/files/container/sample*.html and > /gadgets/files/samplecontainer/samplecontainer.html) to work the same on > both versions. I'm not saying your attribution is way off, however the > wording would almost make one thing it's a huge security hole, while in fact > is is not :-) I hadn't seen this one myself -- why is it even necessary? Couldn't this just be a Directory or RewriteRule? > > > # Server Side Components diagram you list opensocial-0.7 depending on > 'caja', this was actually removed a while ago, since it broke some gadgets > and isn't completed yet. Nothing that would force an update of the graphics, > it's not that important, but still a detail worth mentioning :) > > # pluggable classes. I'll leave it up to you to judge if this is relevant > or not but a lot of the services can be re-configured to your own classes, > things like the gadget blacklist, remote content fetcher & caching classes. > The idea is that a lot of large sites will already have their own versions > that they want to use, so we made all of those configurable (this is the > same in Java land btw). But i guess that would make this document to > complicated? > > Over all it looks great and it's very informative, thanks for your work! > > -- Chris > > > On Jun 25, 2008, at 6:44 PM, Rajdeep Dua wrote: > > Hi Alll, >> I have written an Article on Shindig PHP Architecture, available at the >> following location >> >> http://rajdeep.dua.googlepages.com/shindigPHP-overview-tidy_img.html >> >> Kindly provide feedback and if it can be linked from the Apache site >> >> Thanks >> Rajdeep >> > >
