Refactoring caching utils for api using fastapi-cache (#420)

[peterdudfield]

Pull Request

Description

• use fast-api caching library
• TODO, remove user from key, but make sure other params are in the cache
• TODO, old cache, has a way of locking cache if the route is already in process

#399

How Has This Been Tested?

• CI tests
• ran against local database, and check cache was working
• ran 4 same command `` at the same time and average response time was 15

Checklist:

• My code follows OCF's coding style guidelines
• I have performed a self-review of my own code
• I have made corresponding changes to the documentation
• I have added tests that prove my fix is effective or that my feature works
• I have checked my code and corrected any misspellings

[peterdudfield]

@yuvraajnarula ive left a TODO to do with making sure the user is not in the cache, but other parameters are. Would you be intereted in doing this?

[yuvraajnarula]

yes, I'll be interested in working on it

[yuvraajnarula]

Hi, @peterdudfield.
I removed the user from cache(other params are present) and refactored database.py using type hints.

[peterdudfield]

Hi, @peterdudfield. I removed the user from cache(other params are present) and refactored database.py using type hints.

Thanks, do you mind, doing separate PRs for these different issues. Just so things dont get confused

[yuvraajnarula]

Hi, @peterdudfield. I removed the user from cache(other params are present) and refactored database.py using type hints.

Thanks, do you mind, doing separate PRs for these different issues. Just so things dont get confused

@peterdudfield I've done exactly as you said #429 for removing user from key, but making sure other params are in the cache, and #432 for database.py type hints support.