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
AppRole backend #1426
AppRole backend #1426
Conversation
// Lock to make changes to registered Apps | ||
appLock *sync.RWMutex | ||
|
||
// Lock to make changes to registered Apps |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"to registered Groups"
0e549b4
to
dc234e5
Compare
b401096
to
b1549d8
Compare
3a1eb7a
to
35bd5cd
Compare
HelpDescription: strings.TrimSpace(roleHelp["role-bound-cidr-list"][1]), | ||
}, | ||
&framework.Path{ | ||
Pattern: "role/" + framework.GenericNameRegex("role_name") + "/bound-secret-id$", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
s/bound/bind
51b907a
to
553756e
Compare
553756e
to
afc06a5
Compare
afc06a5
to
59930fd
Compare
the client, it is referred as a `Push` mode. | ||
|
||
While the `user_id` of the App-ID backend worked in a `Push` mode, this backend | ||
recommends the `Pull` mode. The `Pull` mode is supported in AppRole backend, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@vishalnayak shouldn't this say "The Push
mode is supported in AppRole backend only to be able to…"?
Or have I totally misunderstood things? :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@sandstrom, it says that in the master
branch :-)
No description provided.