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

dcache-xrootd: Fix login handshake to support xrootd clients (> 4.7.0) #3562

Merged
merged 1 commit into from Oct 10, 2017

Conversation

vingar
Copy link
Collaborator

@vingar vingar commented Sep 28, 2017

Motivation:

In the 4.7 releases, xrootd client started enforcing protocol requirements
for kXR_login which, unfortunately, broke access to dCache.
xrootd client expects an answer with a 16-character session ID from the door and then
the pool after the redirection.

The login spec can be found at: http://xrootd.org/doc/dev45/XRdv310.htm#_Toc464248819

Without it, the 4.7.0 client declared an error and retries after a while. This will also
fail and so we go into what appears to be a hang. Eventually (after several minutes)
the login will fail with an error.

Modification:

The login answer following the spec has been added in the xrootd pool.

Result:

It works for both xrootd 4.6.1 and 4.7.0.

Target: master
Require-notes: yes
Require-book: no
Patch: https://rb.dcache.org/r/10510
Committed: master@

Motivation:

In the 4.7 releases, xrootd client started enforcing protocol requirements
for kXR_login which, unfortunately, broke access to dCache.
xrootd client expects an answer with a 16-character session ID from the door and then
the pool after the redirection.

The login spec can be found at: http://xrootd.org/doc/dev45/XRdv310.htm#_Toc464248819

Without it, the 4.7.0 client declared an error and retries after a while. This will also
fail and so we go into what appears to be a hang. Eventually (after several minutes)
the login will fail with an error.

Modification:

The login answer following the spec has been added in the xrootd pool.

Result:

It works for both xrootd 4.6.1 and 4.7.0.

Target: master
Require-notes: yes
Require-book: no
Patch: https://rb.dcache.org/r/10510
Committed: master@
@jstarek jstarek merged commit 40bfd38 into 2.16 Oct 10, 2017
@paulmillar paulmillar deleted the fix/2.16/rb10510 branch December 15, 2017 08:34
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 this pull request may close these issues.

None yet

3 participants