Skip to content

Authentication docs - #1708

Open
chrisbeard wants to merge 3 commits into
bloomberg:gh-pagesfrom
chrisbeard:authn-docs
Open

Authentication docs#1708
chrisbeard wants to merge 3 commits into
bloomberg:gh-pagesfrom
chrisbeard:authn-docs

Conversation

@chrisbeard

Copy link
Copy Markdown
Contributor

The Security page will cover authn, authz, and TLS. The initial authn docs cover the basics: the feature set, the protocol, client configuration, and broker plugin/configuration.

Broker-to-broker authn, authz, and TLS docs will be added when the features are fully ready.

Authenticator plugin essentials are also documented in the Plugins page, which we may want to revisit as we expand with additional plugin types, built-in vs external plugins, etc.

The Security page will cover authn, authz, and TLS. The initial authn
docs cover the basics: the feature set, the protocol, client
configuration, and broker plugin/configuration.

Broker-to-broker authn, authz, and TLS docs will be added when the
features are fully ready.

Signed-off-by: Christopher Beard <cbeard9@bloomberg.net>
This commit introduces authenticator plugins to the plugin docs, which
now covers mulitple plugin types.

Signed-off-by: Christopher Beard <cbeard9@bloomberg.net>
Comment thread docs/features/plugins.md Outdated
Comment thread docs/features/plugins.md Outdated
Comment thread docs/features/security.md
connection (see [Failure Handling](#failure-handling) below).

5. If `lifetimeMs` is present, the client SDK schedules reauthentication at
**90 % of the lifetime** so authentication is renewed before expiry.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I am now curious do we have a good reason to have 90% in SDK and 80% in broker-broker

const bsls::Types::Int64 reauthMs = static_cast<bsls::Types::Int64>(
static_cast<double>(lifetimeMs) * 0.8);

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

It doesn't really matter, but we can normalize to 90%.

Comment thread docs/features/security.md

| Field | Description |
|-------|-------------|
| `authenticators` | List of authenticator plugin configurations. Each entry names a plugin and provides its settings. All plugins must have unique mechanisms. |

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Suggested change
| `authenticators` | List of authenticator plugin configurations. Each entry names a plugin and provides its settings. All plugins must have unique mechanisms. |
| `authenticators` | List of authenticator plugin configurations. Each entry names a plugin and provides its settings. All plugins must have unique mechanisms, or the broker will refuse to start. |

Might mention consequences.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Added this to the top, it's generally true for all plugins.

Comment thread docs/features/plugins.md Outdated
Comment thread docs/features/plugins.md Outdated
Signed-off-by: Christopher Beard <cbeard9@bloomberg.net>
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.

2 participants