Skip to content
This repository has been archived by the owner on Feb 13, 2019. It is now read-only.

Vault client caching #133

Closed
wants to merge 7 commits into from
Closed

Vault client caching #133

wants to merge 7 commits into from

Conversation

mparent61
Copy link
Contributor

@collin this is my thought on vault caching, still needs tests + such

@mparent61 mparent61 changed the title Initial implementation Vault caching Mar 3, 2016
@mparent61 mparent61 changed the title Vault caching Vault client caching Mar 3, 2016
@mparent61
Copy link
Contributor Author

@collin @MichaelButkovic @benghaziboy

Suggestion for Vault REST endpoint caching, what do you think?

Idea is to add 2 layers of caching:

  • "short-term" caching to avoid overwhelming Vault service (~1 min)
  • "backstop" caching (I made up the name) to fallback on if Vault endpoint is temporarily unreachable (~60 min)

Short-term seems like a no-brainer, the "backstop" concept could use some feedback.

@camsom
Copy link
Contributor

camsom commented Mar 3, 2016

niiiiiiice

@MelizzaP MelizzaP closed this Mar 16, 2016
@mparent61 mparent61 mentioned this pull request Mar 16, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants