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

Add support for DAVS-style URLs. #1123

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

bbockelm
Copy link
Contributor

This allows the FTS backend to be used for HTTPS-based transfers.

This allows the FTS backend to be used for HTTPS-based transfers;
in the case the DAVS protocol is used, a SciTokens provider is
specified.
Copy link
Contributor

@nataliaratnikova nataliaratnikova left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi,
are there any other commits that go with this one?

@nataliaratnikova
Copy link
Contributor

Found another PR including the same change: #1114
Looks good

@bbockelm
Copy link
Contributor Author

I was going to close out #1114 - decided to wait on the SciTokens support for now and just do the basics.

@nataliaratnikova
Copy link
Contributor

I do not know when next Agents release is out, but I assume the functionality testing will be due at that time, if we decide to included the changes. For now I just double-checked that both changes compile, so it should be no harm.

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.

2 participants