My branch adds the config switch to enable on-demand profiling, and some code that runs the on-demand profile test. However, getting Zope to skip the ++profile++ part of the URL is a problem. We do this for ++oops++, but Gary says that it is a hack, and not something we should copy. So the next step is to ask Gary about a better way to do traversal skips like this.
We might also want ++profile++ to become a real view - one that not only generates the profile, but shows you a helpful page telling you where to get it (or that lets you download it directly).
Ok, the lsprof branch has landed. That branch enables us to turn profiling on and off for all requests at once.
I have started a new branch to add on-demand profiling support for the ++profile++ URL:
lp:~mars/launchpad/lsprof-on-demand
My branch adds the config switch to enable on-demand profiling, and some code that runs the on-demand profile test. However, getting Zope to skip the ++profile++ part of the URL is a problem. We do this for ++oops++, but Gary says that it is a hack, and not something we should copy. So the next step is to ask Gary about a better way to do traversal skips like this.
We might also want ++profile++ to become a real view - one that not only generates the profile, but shows you a helpful page telling you where to get it (or that lets you download it directly).