Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

http redirector and browser woes #368

Closed
osschar opened this issue May 18, 2016 · 5 comments · Fixed by #369
Closed

http redirector and browser woes #368

osschar opened this issue May 18, 2016 · 5 comments · Fixed by #369

Comments

@osschar
Copy link
Contributor

osschar commented May 18, 2016

  1. Accessing paths on a redirector through http (browser) that do not exist in the federation get interpreted as local paths. Having export / on a redirector thus leads to being able to accessing any file on the redirector. I believe redirector should never fall back to local file system.
  2. The generated html includes too many slashes when a directory is the last thing in the url and ends with a /. Also, if one access a server as http://server//foo/bar (note doulbe // after server) the links to files are broken, they point to 'foo/bar/file' (note, all slashes stripped).
@xrootd-dev
Copy link

Hi,

are you referring to XrdHttp or something else ?

Fabrizio

Il 5/18/16 21:13, Matevž Tadel ha scritto:

Accessing paths on a redirector through http (browser) that do not
exist in the federation get interpreted as local paths. Having
export / on a redirector thus leads to being able to accessing any
file on the redirector. I believe redirector should never fall back
to local file system.
The generated html includes too many slashes when a directory is the
last thing in the url and ends with a /. Also, if one access a
server as http://server//foo/bar (note doulbe // after server) the
links to files are broken, they point to 'foo/bar/file' (note, all
slashes stripped).


You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
#368


Use REPLY-ALL to reply to list

To unsubscribe from the XROOTD-DEV list, click the following link:
https://listserv.slac.stanford.edu/cgi-bin/wa?SUBED1=XROOTD-DEV&A=1

@osschar
Copy link
Contributor Author

osschar commented May 19, 2016

Hi Fabrizio,

Yes, I think so :) But I don't know the details of how the things are connected under the hatch.

Matevz

@ffurano
Copy link
Contributor

ffurano commented May 20, 2016

Any chance to have a redirector debug log for point 1 ?

@ffurano
Copy link
Contributor

ffurano commented May 20, 2016

Instead I see the issue with the embedded HTML generation (point 2). I'll have a look at it.

@osschar
Copy link
Contributor Author

osschar commented May 20, 2016

Thanks for fixing issue #2!

I tried reproducing #1 with my own setup (redir exporting / and a single server exporting /tmp) and it indeed works as its should.

I believe the setup I was playing with earlier (and had no control over) had one server with all.export / and oss.localroot /. Sigh, sorry for misdirection.

@abh3 abh3 closed this as completed in #369 May 23, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants