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

Implement namespace issuer auto-discovery #111

Closed
bbockelm opened this issue Sep 7, 2023 · 2 comments · Fixed by #583
Closed

Implement namespace issuer auto-discovery #111

bbockelm opened this issue Sep 7, 2023 · 2 comments · Fixed by #583
Assignees
@jhiemstrawisc
Labels
critical High priority for next release enhancement New feature or request
Milestone
v7.5.0

Comments

@bbockelm
Copy link
Collaborator

bbockelm commented Sep 7, 2023

Currently, when the director wants to download the public keys for an origin, it queries the namespace issuer directly. See: https://github.com/PelicanPlatform/pelican/blob/main/director/origin_api.go#L139-L150

Instead, we should implement "openid-configuration"-style auto-discovery: revisit the code around here to reimplement .well-known-based lookup while keeping backward compatibility -- the current URL can still serve up the public keys.
@bbockelm bbockelm added the enhancement New feature or request label Sep 7, 2023
@bbockelm bbockelm added this to the v7.1.0 milestone Sep 7, 2023
@bbockelm
Copy link
Collaborator Author

bbockelm commented Oct 1, 2023

Since this one is purely internal - and we're running low on v7.1.0 time, I'm pushing this to the v7.2.0 milestone.

@bbockelm bbockelm modified the milestones: v7.1.0, v7.2.0 Oct 1, 2023
@bbockelm bbockelm modified the milestones: v7.2.0, v7.4.0 Oct 29, 2023
@bbockelm bbockelm modified the milestones: v7.4.0, v7.5.0 Dec 29, 2023
@bbockelm
Copy link
Collaborator Author

@jhiemstrawisc - since this has repeatedly gotten pushed back, I'd like to prioritize it for 7.5.0.

@jhiemstrawisc jhiemstrawisc mentioned this issue Jan 3, 2024
Merged
@jhiemstrawisc jhiemstrawisc linked a pull request Jan 3, 2024 that will close this issue
Update ns key discovery to follow openid-style lookups #583
Merged
@jhiemstrawisc jhiemstrawisc added the critical High priority for next release label Jan 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jhiemstrawisc jhiemstrawisc

Labels
critical High priority for next release enhancement New feature or request
Projects
None yet
Milestone
v7.5.0
Development

Successfully merging a pull request may close this issue.

Update ns key discovery to follow openid-style lookups jhiemstrawisc/pelican
2 participants
@bbockelm @jhiemstrawisc