Comment 5 for bug 598289

Revision history for this message
Māris Fogels (mars) wrote :

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