Skip to content

Commit

Permalink
Protocols: Use https:// instead of davs:// rucio#7386
Browse files Browse the repository at this point in the history
This is an ugly workaround, but offers a clear performance benefit. If
ever it proves to be problematic for some RSEs, it can be removed.
  • Loading branch information
dchristidis authored and bari12 committed Feb 10, 2025
1 parent d3c1c0c commit 6b7176e
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion lib/rucio/rse/protocols/gfal.py
Original file line number Diff line number Diff line change
Expand Up @@ -489,7 +489,10 @@ def __gfal2_rm(self, paths):

try:
for path in paths:
ret = ctx.unlink(str(path))
# GFAL does a PROPFIND request before DELETE when the scheme is
# davs://, which is wasteful.
path = re.sub('^davs://', 'https://', str(path))
ret = ctx.unlink(path)
if ret:
return ret
return ret
Expand Down

0 comments on commit 6b7176e

Please sign in to comment.