-
Notifications
You must be signed in to change notification settings - Fork 24.7k
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
Update role-mapping-resources.asciidoc #110451
base: main
Are you sure you want to change the base?
Conversation
used the suggestions from #110420 (comment) to be even more clear.
Documentation preview: |
@jalogisch please enable the option "Allow edits and access to secrets by maintainers" on your PR. For more information, see the documentation. |
Pinging @elastic/es-docs (Team:Docs) |
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.
thanks for following up on this! I've added a couple of suggestions to polish this up even further :)
@@ -70,7 +70,11 @@ example, `"username": "jsmith"`. | |||
`groups`:: | |||
(array of strings) The groups to which the user belongs. For example, `"groups" : [ "cn=admin,ou=groups,dc=example,dc=com","cn=esusers,ou=groups,dc=example,dc=com ]`. | |||
`metadata`:: | |||
(object) Additional metadata for the user. This can include a variety of key-value pairs. When referencing metadata fields in role mapping rules, use the dot notation to specify the key within the metadata object. If the key contains special characters such as parentheses, dots, or spaces, you must escape these characters using backslashes (`\`). For example, `"metadata": { "cn": "John Smith" }`. | |||
|
|||
(object) Additional metadata for the user is passed as key-value pairs. For example, `"metadata": { "cn": "John Smith" }`. |
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.
this structure makes it clearer that "passed as kv pairs" clarifies "additional metadata"
(object) Additional metadata for the user is passed as key-value pairs. For example, `"metadata": { "cn": "John Smith" }`. | |
(object) Additional metadata for the user, passed as key-value pairs. For example, `"metadata": { "cn": "John Smith" }`. |
|
||
When referencing metadata fields in role mapping rules, use dot notation to specify the key within the metadata object. If the key contains special characters such as parentheses, dots, or spaces, you must escape these characters using backslashes (`\`). |
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.
I think we can add a +
to get this formatted so it's indented (it's clearer the content is all related if it's indented)
When referencing metadata fields in role mapping rules, use dot notation to specify the key within the metadata object. If the key contains special characters such as parentheses, dots, or spaces, you must escape these characters using backslashes (`\`). | |
+ | |
When referencing metadata fields in role mapping rules, use dot notation to specify the key within the metadata object. If the key contains special characters such as parentheses, dots, or spaces, you must escape these characters using backslashes (`\`). |
I used the suggestions from #110420 (comment) to make it even clearer.