On 11/22/13 2:56 PM, Daniel Shahaf wrote: > Might not be a bad idea then to make the artificial/invalid > request_rec.filename value look less like a URL then? Just in case it > ends up in someone's "(gdb) p" output, or in a log file, etc. > > For example, "svn+invalid:/usr/local/svn/ppt/trunk".
My primary goal was to make it unlikely to appear as though it was an actual path on the local file system since it really isn't. This is also following a similar pattern in Apache httpd that the proxy module uses where it prefixes the URLs it's proxying to in r->filename as proxy: The lack of the extra slash "svn:/" vs "svn://" should be enough to signal it isn't a URL. I'd be concerned that "svn+invalid:" would imply to people there is something invalid about the path. Ultimately I don't think whatever we put in there is going to make everyone happy. But since quite a few modules crash if r->filename is NULL we had to put something in there, might as well make it somewhat useful.