Problem/Motivation

The module currently assumes that all stored leases are renewable. This may not always be the case. Non renewable leases throw a client error when a renewal is attempted.

Steps to reproduce

Obtain dynamic lease from vault_aws_kv using a federated token.

Proposed resolution

Add the ability to specify if a lease is renewable when storing. Check renewal flag before attempting renewal.

Remaining tasks

User interface changes

API changes

Client will now accept if lease is renewable (default to TRUE for BC)
Lease Storage must now accept a renewable field.

Data model changes

Lease storage plugins will now need to store a renewable flag.

Issue fork vault-3486023

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

cmlara created an issue. See original summary.

cmlara’s picture

Version: 2.x-dev » 3.x-dev

cmlara’s picture

Status: Active » Fixed

  • cmlara committed 3b667d97 on 3.x
    Issue #3486023 by cmlara: Allow stored leases to specify renewable...

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.