search

benluteijn / cherokee

Automatically exported from code.google.com/p/cherokee
0 stars 1 forks source link

Symlinks in List and Send doesn't resolve a directory #193

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
What steps will reproduce the problem?
1. create a link to a directory
2. directory is not resolved when clicking on it
3. it is resolved when browsing to it

Original issue reported on code.google.com by ste...@konink.de on 31 Oct 2008 at 10:09

GoogleCodeExporter commented 9 years ago 
It is odd. It doesn't happen all the time.

I have configured:
/kabelkrant as directory that refers to /opt/cherokee/var/www/broadcast

In the directory broadcast I did:
ln -sf . broadcast

Like on other cherokee installs I see:
http://88.159.247.133/kabelkrant/broadcast

But when I click on it I'll get:
http://88.159.247.133/kabelkranthttp://88.159.247.133/broadcast/

Original comment by ste...@konink.de on 31 Oct 2008 at 11:21

GoogleCodeExporter commented 9 years ago 
Ok can reproduce it;

A symlink inside a 'virtual directory' has this behavior. The patch essentially 
fixes
this behavior, but I wonder what causes this in the first place.

After *many* hours of searching and one try of wireshark:

cherokee_connection_set_redirect <- b0rk3d

I'll try to fix the cause of it.

Alvaro: could you find out what my last patch was that went into handler_dirlist
(long ago)? I suspect it might is a fix for the same thing.

Original comment by ste...@konink.de on 1 Nov 2008 at 1:51

Attachments:

GoogleCodeExporter commented 9 years ago 
I suggest to use the *above* to prevent the redir from happing when happy 
browsing.
(then ignore my comment)

And use this patch to solve the problem :)

Original comment by ste...@konink.de on 1 Nov 2008 at 1:57

Attachments:

GoogleCodeExporter commented 9 years ago 
Ey, Stefan.. good catch!! :-)

I've changed something though.  The block you moved should be after the userdir 
inclusion.  Check it at:  http://svn.cherokee-project.com/changeset/2276

Original comment by alobbs on 1 Nov 2008 at 8:30

GoogleCodeExporter commented 9 years ago 
The rest has been also committed at r2279:

  http://svn.cherokee-project.com/changeset/2279

Good job! :-)

Original comment by alobbs on 1 Nov 2008 at 9:33