Skip to content

Update vault and redis APIs as suggested in deprecation warnings. fix #950#954

Merged
pedro-psb merged 2 commits intodynaconf:masterfrom
pedro-psb:fix-hvac-deprecation-warning
Jul 11, 2023
Merged

Update vault and redis APIs as suggested in deprecation warnings. fix #950#954
pedro-psb merged 2 commits intodynaconf:masterfrom
pedro-psb:fix-hvac-deprecation-warning

Conversation

@pedro-psb
Copy link
Copy Markdown
Member

Vault

The deprecation warning was quite straighfoward. Only the kv2.read_secret_version seems to be involved.

Redis

This explain the redis deprecation warning and this shows that hmset can be replaced by hset. The use of hset here requires redis v4 (2015) or higher. Should we add a fallback?

@netlify
Copy link
Copy Markdown

netlify bot commented Jul 10, 2023

Deploy Preview for dynaconf ready!

Name Link
🔨 Latest commit a8a8bf7
🔍 Latest deploy log https://app.netlify.com/sites/dynaconf/deploys/64ad85f58f12b400083895ff
😎 Deploy Preview https://deploy-preview-954--dynaconf.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@rochacbruno
Copy link
Copy Markdown
Member

we are going to release 3.2.0 so I am ok not having a fallbak

@pedro-psb pedro-psb merged commit 22690fe into dynaconf:master Jul 11, 2023
@pedro-psb pedro-psb deleted the fix-hvac-deprecation-warning branch July 11, 2023 17:02
pedro-psb added a commit to pedro-psb/dynaconf that referenced this pull request Jul 13, 2023
Shortlog of commits since last release:

    Bruno Rocha (4):
          Ignore docs build without a tag
          Cancel any running CI job when a Push is made to an existing PR or branch (dynaconf#952)
          Fix dynaconf#959 cli get will exit code 1 in case of KeyError. (dynaconf#960)
          add tech preview note to inspect docs (dynaconf#961)

    Pedro Pessoa (9):
          Docs - Update envvar.md custom token e.g. to use add_converter (dynaconf#941)
          Feature - Inspect and CLI (dynaconf#939)
          Fix - Template substitution with variable update (dynaconf#944)
          Assert dynaconf#658 works (dynaconf#945)
          fix infinite recursions in special case of django app dynaconf#867 (dynaconf#947)
          Fix - Django functions with `add_converter` (dynaconf#951)
          Fix hooks not re-running on reload dynaconf#850 (dynaconf#953)
          update vault and redis warning recommendations. fix dynaconf#950 (dynaconf#954)
          Fix - Enable merge equal False (dynaconf#957)
rochacbruno added a commit that referenced this pull request Aug 11, 2023
Shortlog of commits since last release:

    Bruno Rocha (5):
          Ignore docs build without a tag
          Cancel any running CI job when a Push is made to an existing PR or branch (#952)
          Fix #959 cli get will exit code 1 in case of KeyError. (#960)
          add tech preview note to inspect docs (#961)
          Build docs

    Hugo Prudente (1):
          Doc advanced usage for cli overrides dynaconf settings fix #967 (#970)

    Marian Ganisin (1):
          Feat: Support for multidoc yaml files (#825)

    Pedro Pessoa (11):
          Docs - Update envvar.md custom token e.g. to use add_converter (#941)
          Feature - Inspect and CLI (#939)
          Fix - Template substitution with variable update (#944)
          Assert #658 works (#945)
          fix infinite recursions in special case of django app #867 (#947)
          Fix - Django functions with `add_converter` (#951)
          Fix hooks not re-running on reload #850 (#953)
          update vault and redis warning recommendations. fix #950 (#954)
          Fix - Enable merge equal False (#957)
          CI - Test docker-compose pyyaml issue (#964)
          Fix: unexpected _bypass_evaluation in BoxList (#966)

    pedro-psb (1):
          Release version 3.2.0
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