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

Harvest should support StorageGRID Azure AD SSO auth #2049

Closed
cgrinds opened this issue May 5, 2023 · 3 comments · Fixed by #3048
Closed

Harvest should support StorageGRID Azure AD SSO auth #2049

cgrinds opened this issue May 5, 2023 · 3 comments · Fixed by #3048

Comments

@cgrinds
Copy link
Collaborator

cgrinds commented May 5, 2023

hello @cgrinds

Thanks a ton for your quick response. Apologies if my last question was confusing. So when the SSO was disabled we were able to make it work using local users but with the new security standard we need to use SSO on our Storagegrid System.

The link I provided helps us get a "token" which we can use for the api operations. So was hoping once you get the token rest of the harvest related operations should be same.

Do I need to raise it as an issue for the feature request ?

Originally posted by @santosh725 in #2048 (comment)

@cgrinds cgrinds added the 23.08 label May 12, 2023
@cgrinds
Copy link
Collaborator Author

cgrinds commented Jul 28, 2023

See latest comment.

Moving out of 23.08

@cgrinds
Copy link
Collaborator Author

cgrinds commented Jul 15, 2024

With change #3048, Harvest's StorageGRID collector will use authTokens returned from credential scripts to authenticate with StorageGRID.

To write a credential script use StorageGRID's provided script.

@cgrinds
Copy link
Collaborator Author

cgrinds commented Aug 8, 2024

Verified on 24.08.0 commit 0d2d7b4
with unit test and made sure that authToken, returned from script, is set as the Bearer token in the request header for the StorageGRID client.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant