Move lease renewal to LeaseStorage plugins.

Created on 19 March 2023, almost 2 years ago

Updated 22 March 2023, almost 2 years ago

Problem/Motivation

Currently we have a public getAllLeases() method available on LeaseStorage plugins to facilitate renewAllLeases(), this provides a public method that reveals all the currently stored leases.

Steps to reproduce

Review

Proposed resolution

Move renewal inside the LeaseStorage plugins so that we can make this a protected internal method which will increase the complexity for an attacker obtaining leases.

Remaining tasks

Patch

User interface changes

None

API changes

Remove getAllLeases() from VaultLeaseStorageInterface(). It shall remain an abstract method in VaultLeaseStoragePluginBase to be implemented by extenders.
A no-lease-storage client will be required for all LeaseStorage plugins which will create the lease.

Data model changes

None

📌 Task

Status

Fixed

Version

2.0

Component

Code

Created by

🇺🇸United States cmlara

Live updates comments and jobs are added and updated live.

Comments & Activities

Issue created by @cmlara

Comment almost 2 years ago →

System Message

cmlara → committed df3af662 on 2.x

Issue #3348899: Move lease renewal to LeaseStorage plugins

The primary...

Status changed to Fixed almost 2 years ago10:40pm 21 March 2023
Comment almost 2 years ago →
🇺🇸United States cmlara
Issue was unassigned.
Comment almost 2 years ago →
🇺🇸United States cmlara
Comment almost 2 years ago →
System Message
Automatically closed - issue fixed for 2 weeks with no activity.

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024