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

[XrdCrypto][XrdHttp] Extract DN from user (proxy, multi-proxy) certif… #1238

Merged
merged 2 commits into from
Jul 10, 2020

Conversation

esindril
Copy link
Contributor

@esindril esindril commented Jul 2, 2020

…icate and properly

handle the gridmap-file functionality when accessing through HTTP

This pull request is aimed at R5.

@esindril esindril requested a review from abh3 July 2, 2020 15:45
@esindril
Copy link
Contributor Author

esindril commented Jul 2, 2020

There is still the issue that if the certificate is not retrieved and we fail then no token (macaroon, scitoken etc) access is possible. For the moment this is allowed, therefore tokens work with the PR. This happens here:
https://github.com/xrootd/xrootd/pull/1238/files#diff-7b19b9632974132e3ff4f2f2f1924e1aR108

bbockelm and others added 2 commits July 8, 2020 22:57
…icate and properly

  handle the gridmap-file functionality when accessing through HTTP
@esindril
Copy link
Contributor Author

esindril commented Jul 8, 2020

Hi Andy,

This is now ready from my side. Note this PR is done against the current master.

Thanks,
Elvin

@abh3
Copy link
Member

abh3 commented Jul 8, 2020 via email

@esindril
Copy link
Contributor Author

esindril commented Jul 8, 2020

I am preparing that one now and this will be in #1239 done against the stable-4.12.x branch.

@abh3 abh3 merged commit e76af17 into xrootd:master Jul 10, 2020
@esindril esindril deleted the xrdhttp_proxy_handling_v5 branch February 7, 2023 15:16
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