Skip to content

Backport 1.5.4: Don't create/overwrite raft snapshot output file on error#9944

Merged
sgmiller merged 1 commit into
release/1.5.xfrom
backport-pr-9894-1.5.x
Sep 14, 2020
Merged

Backport 1.5.4: Don't create/overwrite raft snapshot output file on error#9944
sgmiller merged 1 commit into
release/1.5.xfrom
backport-pr-9894-1.5.x

Conversation

@sgmiller
Copy link
Copy Markdown
Collaborator

No description provided.

…esponse. (#9894)

* Don't open the snapshot file until we have a successful response

* Check the success of Close if nothing else errors
@sgmiller sgmiller changed the base branch from master to release/1.5.x September 14, 2020 17:26
@sgmiller sgmiller requested a review from ncabatoff September 14, 2020 17:26
@sgmiller sgmiller merged commit c0f1327 into release/1.5.x Sep 14, 2020
@sgmiller sgmiller deleted the backport-pr-9894-1.5.x branch September 14, 2020 18:22
pull Bot pushed a commit to mayocream/vault that referenced this pull request Oct 10, 2025
…ashicorp#9944) (hashicorp#10070)

* normalize token type for ns_token auth mounts

* add changelog

* also check type is supported in route and add test coverage

* Apply suggestion from @hellobontempo

* update test coverage to expect array

* update tests

Co-authored-by: claire bontempo <68122737+hellobontempo@users.noreply.github.com>
tsaarni pushed a commit to Nordix/vault that referenced this pull request Oct 23, 2025
…ashicorp#9944) (hashicorp#10073) (hashicorp#10080)

* normalize token type for ns_token auth mounts

* add changelog

* also check type is supported in route and add test coverage

* Apply suggestion from @hellobontempo

* update test coverage to expect array

* update tests

Co-authored-by: claire bontempo <68122737+hellobontempo@users.noreply.github.com>
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