Skip to content

Commit fef66e6

Browse files
Apply suggestions from code review
Co-authored-by: Lukas Krug <lukas.voetmand@stackable.tech>
1 parent 2ef2dc0 commit fef66e6

2 files changed

Lines changed: 4 additions & 4 deletions

File tree

CHANGELOG.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ All notable changes to this project will be documented in this file.
66

77
### Changed
88

9-
- BREAKING: Reworked authorization config to closer match the Apache NiFi internal autorizer interfaces ([#884]).
9+
- BREAKING: Reworked authorization config to closer match the Apache NiFi internal authorizer interfaces ([#884]).
1010

1111
### Fixed
1212

rust/operator-binary/src/security/authorization.rs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -142,7 +142,7 @@ impl ResolvedNifiAuthorizationConfig {
142142
ResolvedNifiAuthorizationConfig::Standard {
143143
access_policy_provider: NifiAccessPolicyProvider::FileBased { initial_admin_user },
144144
} => {
145-
let file_based_mount_path = Self::filebased_mount_path();
145+
let file_based_mount_path = Self::file_based_mount_path();
146146

147147
authorizers_xml.push_str(&formatdoc! {r#"
148148
<userGroupProvider>
@@ -230,7 +230,7 @@ impl ResolvedNifiAuthorizationConfig {
230230
) {
231231
volume_mounts.push(VolumeMount {
232232
name: FILE_BASED_MOUNT_NAME.into(),
233-
mount_path: Self::filebased_mount_path(),
233+
mount_path: Self::file_based_mount_path(),
234234
..VolumeMount::default()
235235
})
236236
}
@@ -301,7 +301,7 @@ impl ResolvedNifiAuthorizationConfig {
301301
)
302302
}
303303

304-
fn filebased_mount_path() -> String {
304+
fn file_based_mount_path() -> String {
305305
format!("{NIFI_PVC_STORAGE_DIRECTORY}/{FILE_BASED_MOUNT_DIRECTORY}")
306306
}
307307
}

0 commit comments

Comments
 (0)