Skip to content
GitLab
Menu
Projects
Groups
Snippets
Help
Projects
Groups
Snippets
Help
Help
Support
Community forum
Keyboard shortcuts
?
Submit feedback
Sign in
Toggle navigation
Menu
Open sidebar
小 白蛋
Vault
Commits
34aa9a07
Unverified
Commit
34aa9a07
authored
3 years ago
by
Nick Cabatoff
Committed by
GitHub
3 years ago
Browse files
Options
Download
Email Patches
Plain Diff
Fix regression in returning empty value for approle cidrlist. (#13235) (#13251)
parent
3b8ac395
Branches unavailable
v1.9.6
v1.9.5
v1.9.4
v1.9.3
v1.9.2
v1.9.1
No related merge requests found
Changes
2
Hide whitespace changes
Inline
Side-by-side
Showing
2 changed files
builtin/credential/approle/validation.go
+5
-0
builtin/credential/approle/validation.go
changelog/13235.txt
+3
-0
changelog/13235.txt
with
8 additions
and
0 deletions
+8
-0
builtin/credential/approle/validation.go
+
5
-
0
View file @
34aa9a07
...
...
@@ -118,6 +118,11 @@ func decodeSecretIDStorageEntry(entry *logical.StorageEntry) (*secretIDStorageEn
}
cleanup
:=
func
(
in
[]
string
)
[]
string
{
if
len
(
in
)
==
0
{
// Don't change unnecessarily, if it was empty list leave as empty list
// instead of making it nil.
return
in
}
var
out
[]
string
for
_
,
s
:=
range
in
{
out
=
append
(
out
,
parseip
.
TrimLeadingZeroesCIDR
(
s
))
...
...
This diff is collapsed.
Click to expand it.
changelog/13235.txt
0 → 100644
+
3
-
0
View file @
34aa9a07
```release-note:bug
auth/approle: Fix regression where unset cidrlist is returned as nil instead of zero-length array.
```
\ No newline at end of file
This diff is collapsed.
Click to expand it.
Write
Preview
Supports
Markdown
0%
Try again
or
attach a new file
.
Attach a file
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Cancel
Please
register
or
sign in
to comment
Menu
Projects
Groups
Snippets
Help