Skip to content

Feature managed identity auth#94

Merged
ddlees merged 8 commits intoSpecterOps:mainfrom
daviditkin:feature-managed-identity-auth
Feb 25, 2025
Merged

Feature managed identity auth#94
ddlees merged 8 commits intoSpecterOps:mainfrom
daviditkin:feature-managed-identity-auth

Conversation

@daviditkin
Copy link
Contributor

conf.json supports the managed-identity boolean property. When true, managed identity authentication will be used.

Refactored authentication logic out of RestClient into Authenticator along with AuthStrategy(s) including ManagedIdentityAuthStrategy. I did this because I didn't want to add more if/else logic to the RestClient than is already there.

GenericAuthStrategy is just the original collection of authentication logic that already existed. It could be broken up into additional AuthStrategy types.

Token was modified to handle either int or string integer json values for the expiration times. See IntOrStringInt.

@daviditkin
Copy link
Contributor Author

I have read the CLA Document and I hereby sign the CLA

@ddlees
Copy link
Member

ddlees commented Dec 17, 2024

recheck

1 similar comment
@ddlees
Copy link
Member

ddlees commented Dec 17, 2024

recheck

@ddlees ddlees merged commit 68d73d5 into SpecterOps:main Feb 25, 2025
9 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Feb 25, 2025
@daviditkin daviditkin deleted the feature-managed-identity-auth branch March 27, 2025 15:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants