Skip to content

Commit fb23f1c

Browse files
committed
chore(stackable-operator): Adjust doc comments
1 parent 782460e commit fb23f1c

File tree

1 file changed

+20
-15
lines changed
  • crates/stackable-operator/src/crd/authentication/core

1 file changed

+20
-15
lines changed

crates/stackable-operator/src/crd/authentication/core/mod.rs

Lines changed: 20 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -83,18 +83,20 @@ pub mod versioned {
8383
Oidc(oidc::v1alpha1::AuthenticationProvider),
8484

8585
/// The [TLS provider](DOCS_BASE_URL_PLACEHOLDER/concepts/authentication#_tls).
86-
/// The TLS AuthenticationClass is used when users should authenticate themselves with a TLS certificate.
86+
/// The TLS AuthenticationClass is used when users should authenticate themselves with a
87+
/// TLS certificate.
8788
Tls(tls::v1alpha1::AuthenticationProvider),
8889

8990
/// The [Kerberos provider](DOCS_BASE_URL_PLACEHOLDER/concepts/authentication#_kerberos).
90-
/// The Kerberos AuthenticationClass is used when users should authenticate themselves via Kerberos.
91+
/// The Kerberos AuthenticationClass is used when users should authenticate themselves via
92+
/// Kerberos.
9193
Kerberos(kerberos::v1alpha1::AuthenticationProvider),
9294
}
9395

94-
/// Common [`v1alpha1::ClientAuthenticationDetails`] which is specified at the client/ product
95-
/// cluster level. It provides a name (key) to resolve a particular [`AuthenticationClass`].
96-
/// Additionally, it provides authentication provider specific configuration (OIDC and LDAP for
97-
/// example).
96+
/// Common client authentication details which is specified at the client/ product cluster level.
97+
///
98+
/// It provides a name (key) to resolve a particular [`AuthenticationClass`]. Additionally, it
99+
/// provides authentication provider specific configuration (OIDC and LDAP for example).
98100
///
99101
/// If the product needs additional (product specific) authentication options, it is recommended
100102
/// to wrap this struct and use `#[serde(flatten)]` on the field.
@@ -127,19 +129,22 @@ pub mod versioned {
127129
pub struct ClientAuthenticationDetails<O = ()> {
128130
/// Name of the [AuthenticationClass](https://docs.stackable.tech/home/nightly/concepts/authentication) used to
129131
/// authenticate users.
130-
//
131-
// To get the concrete [`AuthenticationClass`], we must resolve it. This resolution can be achieved by using
132-
// [`ClientAuthenticationDetails::resolve_class`].
132+
///
133+
/// To get the concrete [`AuthenticationClass`], we must resolve it. This resolution can be
134+
/// achieved by using [`ClientAuthenticationDetails::resolve_class`].
133135
#[serde(rename = "authenticationClass")]
134136
authentication_class_ref: String,
135137

136-
/// This field contains OIDC-specific configuration. It is only required in case OIDC is used.
138+
/// This field contains OIDC-specific configuration. It is only required in case OIDC is
139+
/// used.
140+
///
141+
/// Use [`ClientAuthenticationDetails::oidc_or_error`] to get the value or report an error
142+
/// to the user.
137143
//
138-
// Use [`ClientAuthenticationDetails::oidc_or_error`] to get the value or report an error to the user.
139-
// TODO: Ideally we want this to be an enum once other `ClientAuthenticationOptions` are added, so
140-
// that user can not configure multiple options at the same time (yes we are aware that this makes a
141-
// changing the type of an AuthenticationClass harder).
142-
// This is a non-breaking change though :)
144+
// TODO: Ideally we want this to be an enum once other `ClientAuthenticationOptions` are
145+
// added, so that user can not configure multiple options at the same time (yes we are aware
146+
// that this makes a changing the type of an AuthenticationClass harder). This is a
147+
// non-breaking change though :)
143148
oidc: Option<oidc::v1alpha1::ClientAuthenticationOptions<O>>,
144149
}
145150
}

0 commit comments

Comments
 (0)