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 :-)

# 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

Reply via email to