Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
docs: add security settings and builtin security reference #13319
base: main
Are you sure you want to change the base?
docs: add security settings and builtin security reference #13319
Changes from 16 commits
d34f99e
ddf6eed
802086d
0ac6879
1010946
5719d66
aab402a
b38f26d
b105adf
8ea4022
f863b9a
f2baf54
9816673
d76f100
0e321d2
e907708
0c50ccb
8110d97
951ace1
beccbc2
021dd6f
0276037
328fb46
ff8f8e0
f51c846
ccd9d07
4b8e0af
e5115b7
2696e83
f6e3ef3
e29a849
1588fae
7576c0d
862fc24
b6f6004
b600c3c
db5014a
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
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.
Не согласен: объект доступа и субъект доступа это абстракции, абстракции должны быть связаны друг с другом, но правильнее и удобнее для читателя, когда абстрактные понятия объясняются через простые вещи, а не через друг друга.
Порядок "пользователь (то есть субъект)" лучше чем "субъект (то есть пользователь)".
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.
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.
Права доступа
имеют смысл только для схемных объектов. Вышеобъект доступа
определяется черезсхемный объект
, но вообще говоря это слишком узко. В системе, кроме схемных объектов, ещё полно мест, которые не связаны со схемой, и доступ куда контролируется. Их тоже вообще-то можно было бы назвать "объектами доступа".Поэтому я специально пишу здесь про "контексты, не связанные со схемой" -- так меньше недоумения, чем если бы были
объекты доступа
.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.
Уровня доступа
должна вести на статью, где уровни доступа объясняются. В конечном счёте это должна бытьsecurity/authorization.md
, но в момент создания pr её ещё не было и уровни доступа описываются вreference/configuration/index.md##security-access-levels
. Я сделал rebase своего pr на Сашин и "сшил" материалы новыми терминами и ссылками. Цели сделать полное слияние за один раз нет, будут follow-ups и следующие итерации.Перечисление "других возможностей" для разных уровней доступа -- тоже follow-up, [YDBDOCS-986: Справка по возможностям, добавляемым каждым
*_allowed_sids
].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.
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.
Нет, в pr делается усилие, чтобы, насколько возможно, термин
права доступа
закрепить за ACL на схемных объектах, а терминыразрешения
ивозможности
-- зауровнями доступа
. Специально, чтобы уменьшить путаницу (до нуля всё равно не получится).Будет хорошо, есть появятся ещё мысли по терминам. Но разводить разное по разным терминам -- надо.
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.
В других текстах упоминался суффикс
@builtin
— это синоним к отсутствию суффикса или отличается? Возможно стоит тут про него пару слов сказать.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.
А ниже ещё
@well-known
есть.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.
@builtin
не синоним отсутствию суффикса: нет суффикса -- это реальный локальный пользователь (который должен логиниться и предоставлять в запросах auth-токен), а любая строка с@builtin
-- виртуальный пользователь, чья аутентификация будет всегда успешна.Рассказ про
@builtin
-- тоже follow-up, и в дополнение [YDBDOCS-991: Справка по секции конфигурацииauth_config
], где есть настройка про отключение@builtin
.Рассказ про возможные суффиксы (их же ещё и из конфига можно менять!) -- тоже follow-up, где-то в районе [YDBDOCS-979: Описание модели управления доступом].
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.
Звучит как-будто «первоначальная настройка» уже произошла выше автоматически, а тут скорее какая-то дополнительная настройка т.к. builtin security почему-то не умеет сама пароль для root устанавливать.
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.
Я намеренно не трогаю
initial-deployment.md
, это follow-up.Кусок про настройку безопасности там конечно надо подтягивать.
Builtin security создаёт
root
без пароля, да. Хотя доп.настройками (обоимиdefault_users
) можно было бы добиться, чтобы суперпользователь был с нужным паролем. Но тогда это уже будет полу-builtin настройка безопасности.