Skip to content

feat: add v3 Keystone trust auth support#105

Open
manadart wants to merge 1 commit into
go-goose:v5from
manadart:v5-keystone-trusts
Open

feat: add v3 Keystone trust auth support#105
manadart wants to merge 1 commit into
go-goose:v5from
manadart:v5-keystone-trusts

Conversation

@manadart
Copy link
Copy Markdown
Collaborator

This adds support for Keystone Trusts.

The change is quite simple - a single, optional extra field in the Credentials struct for supplying a trust ID.

Copy link
Copy Markdown
Member

@wallyworld wallyworld left a comment

Choose a reason for hiding this comment

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

just a typo

} `json:"domain"`
Trust struct {
ID string `json:"id"`
} `json:"OS-TRUST:trust"`
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

typo?

Copy link
Copy Markdown
Member

@hmlanigan hmlanigan left a comment

Choose a reason for hiding this comment

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

It may not be possible in this PR, but moving to go 1.26 would be good.

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.

3 participants