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.

Sign up for GitHub

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

Jump to bottom

Make token-support exclusive to WebDAV #6412

Closed
dchristidis opened this issue Dec 11, 2023 · 0 comments · Fixed by #6413
Closed

Make token-support exclusive to WebDAV #6412

dchristidis opened this issue Dec 11, 2023 · 0 comments · Fixed by #6413
Assignees
@dchristidis
Labels
Deletion enhancement Transfers
Milestone
33.1.0

Comments

@dchristidis
Copy link
Contributor

Description

It has been brought to our attention that there are some requirements in XRootD that might make token support more complex.

  1. The scheme must be roots and not root.
  2. The only way to to a token to the XRootD library is through an environmental variable (e.g. BEARER_TOKEN).

It was agreed to focus on WebDAV for now and review XRootD after the Data Challenge 2024.

Motivation

Focusing on a single protocol (thus not having to worry about protocol-specific quirks) should ease early-stage development and debugging.

Change

Make use of tokens for transfers and deletions only if the protocol scheme being used is davs.
@dchristidis dchristidis self-assigned this Dec 11, 2023
@dchristidis dchristidis linked a pull request Dec 11, 2023 that will close this issue
Deletion/Transfers: Make token support WebDAV-only #6412 #6413
Merged
dchristidis added a commit to dchristidis/rucio that referenced this issue Dec 11, 2023
@dchristidis
Deletion/Transfers: Make token support WebDAV-only rucio#6412
993e86f 
There are some requirements in XRootD that might make token support more
complex. It was agreed to focus on WebDAV for now and review XRootD
after the Data Challenge 2024.

This commits also applies a stronger restriction to the content of the
`oidc_support` RSE attribute.
dchristidis added a commit to dchristidis/rucio that referenced this issue Dec 11, 2023
@dchristidis
Deletion/Transfers: Make token support WebDAV-only rucio#6412
8f58869 
There are some requirements in XRootD that might make token support more
complex. It was agreed to focus on WebDAV for now and review XRootD
after the Data Challenge 2024.

This commits also applies a stronger restriction to the content of the
`oidc_support` RSE attribute.
bari12 pushed a commit that referenced this issue Dec 18, 2023
@dchristidis @bari12
Deletion/Transfers: Make token support WebDAV-only #6412
860c88d 
There are some requirements in XRootD that might make token support more
complex. It was agreed to focus on WebDAV for now and review XRootD
after the Data Challenge 2024.

This commits also applies a stronger restriction to the content of the
`oidc_support` RSE attribute.
bari12 pushed a commit that referenced this issue Dec 18, 2023
@dchristidis @bari12
Deletion/Transfers: Make token support WebDAV-only #6412
76cfecf 
There are some requirements in XRootD that might make token support more
complex. It was agreed to focus on WebDAV for now and review XRootD
after the Data Challenge 2024.

This commits also applies a stronger restriction to the content of the
`oidc_support` RSE attribute.
@bari12 bari12 modified the milestones: 32.6.1 / 32.7.0, 33.1.0 Dec 18, 2023
@dchristidis dchristidis mentioned this issue Jan 17, 2024
Closed
voetberg pushed a commit to voetberg/rucio that referenced this issue Apr 15, 2024
@dchristidis @voetberg
Deletion/Transfers: Make token support WebDAV-only rucio#6412
586bb95 
There are some requirements in XRootD that might make token support more
complex. It was agreed to focus on WebDAV for now and review XRootD
after the Data Challenge 2024.

This commits also applies a stronger restriction to the content of the
`oidc_support` RSE attribute.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dchristidis dchristidis

Labels
Deletion enhancement Transfers
Projects
None yet
Milestone
33.1.0
Development

Successfully merging a pull request may close this issue.

Deletion/Transfers: Make token support WebDAV-only #6412 dchristidis/rucio
2 participants
@bari12 @dchristidis