Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

refactor(vault): Refactor vault package #5148

Merged
merged 10 commits into from
Oct 22, 2024
Merged

refactor(vault): Refactor vault package #5148

merged 10 commits into from
Oct 22, 2024

Conversation

Googlom
Copy link
Member

@Googlom Googlom commented Oct 15, 2024

Main point of this refactor is to introduce token reneval by using Vault API recommended way.

Only Client creation is refactored. Methods which actually make API calls are untouched.

@CCFenner CCFenner changed the title (Draft) refactor(vault): Refactor vault package refactor(vault): Refactor vault package Oct 21, 2024
@Googlom Googlom marked this pull request as ready for review October 21, 2024 12:36
@Googlom Googlom requested a review from a team as a code owner October 21, 2024 12:36
pkg/vault/oidc.go Outdated Show resolved Hide resolved
pkg/vault/oidc.go Outdated Show resolved Hide resolved
pkg/vault/oidc.go Outdated Show resolved Hide resolved
pkg/vault/vault.go Outdated Show resolved Hide resolved
pkg/vault/vault.go Outdated Show resolved Hide resolved
Copy link

Copy link
Member

@CCFenner CCFenner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@Googlom
Copy link
Member Author

Googlom commented Oct 22, 2024

/it-go

@Googlom Googlom merged commit 5c47be3 into master Oct 22, 2024
12 checks passed
@Googlom Googlom deleted the vault_refactor branch October 22, 2024 08:29
maxatsap added a commit to maxatsap/jenkins-library that referenced this pull request Nov 20, 2024
…ix-fix

* origin/master: (43 commits)
  fix handle new naming strategy for Central Build stage name if extension is used (SAP#5178)
  Remove commit message from project-metadata.toml (SAP#5176)
  handle new naming strategy for Central Build stage name (SAP#5171)
  Update URL to current SapMachine (SAP#5126)
  checkmarxOneExecuteScan - Fix report generation in CxOne 3.20 (SAP#5170)
  docs: update pr template to include inner source update reminder (SAP#5169)
  handle error while fetching working directory (SAP#5168)
  Final round of adding quoting to prevent command injection (SAP#5167)
  feat: add build artifacts metadata for mtaBuild (SAP#5166)
  Fix more potential command injection via quoting (SAP#5164)
  Add identifier data to create uuid in events (SAP#5165)
  Update version of org.cyclonedx:cyclonedx-maven-plugin (SAP#5156)
  Fix: validate app name (SAP#5155)
  fix: command injection vulnerability (SAP#5161)
  fix deadlock and add more logging (SAP#5160)
  fix(sonar): allign groovy part of Sonar step with common piperExecuteBin (SAP#5157)
  modify logging (SAP#5158)
  Detect script version 9 (SAP#5154)
  feat(vault): not allowing batch token revoke (SAP#4918)
  refactor(vault): Refactor vault package (SAP#5148)
  ...
maxatsap added a commit to maxatsap/jenkins-library that referenced this pull request Nov 20, 2024
* origin/master: (43 commits)
  fix handle new naming strategy for Central Build stage name if extension is used (SAP#5178)
  Remove commit message from project-metadata.toml (SAP#5176)
  handle new naming strategy for Central Build stage name (SAP#5171)
  Update URL to current SapMachine (SAP#5126)
  checkmarxOneExecuteScan - Fix report generation in CxOne 3.20 (SAP#5170)
  docs: update pr template to include inner source update reminder (SAP#5169)
  handle error while fetching working directory (SAP#5168)
  Final round of adding quoting to prevent command injection (SAP#5167)
  feat: add build artifacts metadata for mtaBuild (SAP#5166)
  Fix more potential command injection via quoting (SAP#5164)
  Add identifier data to create uuid in events (SAP#5165)
  Update version of org.cyclonedx:cyclonedx-maven-plugin (SAP#5156)
  Fix: validate app name (SAP#5155)
  fix: command injection vulnerability (SAP#5161)
  fix deadlock and add more logging (SAP#5160)
  fix(sonar): allign groovy part of Sonar step with common piperExecuteBin (SAP#5157)
  modify logging (SAP#5158)
  Detect script version 9 (SAP#5154)
  feat(vault): not allowing batch token revoke (SAP#4918)
  refactor(vault): Refactor vault package (SAP#5148)
  ...
maxatsap added a commit to maxatsap/jenkins-library that referenced this pull request Nov 20, 2024
…fig-fix

* origin/master: (43 commits)
  fix handle new naming strategy for Central Build stage name if extension is used (SAP#5178)
  Remove commit message from project-metadata.toml (SAP#5176)
  handle new naming strategy for Central Build stage name (SAP#5171)
  Update URL to current SapMachine (SAP#5126)
  checkmarxOneExecuteScan - Fix report generation in CxOne 3.20 (SAP#5170)
  docs: update pr template to include inner source update reminder (SAP#5169)
  handle error while fetching working directory (SAP#5168)
  Final round of adding quoting to prevent command injection (SAP#5167)
  feat: add build artifacts metadata for mtaBuild (SAP#5166)
  Fix more potential command injection via quoting (SAP#5164)
  Add identifier data to create uuid in events (SAP#5165)
  Update version of org.cyclonedx:cyclonedx-maven-plugin (SAP#5156)
  Fix: validate app name (SAP#5155)
  fix: command injection vulnerability (SAP#5161)
  fix deadlock and add more logging (SAP#5160)
  fix(sonar): allign groovy part of Sonar step with common piperExecuteBin (SAP#5157)
  modify logging (SAP#5158)
  Detect script version 9 (SAP#5154)
  feat(vault): not allowing batch token revoke (SAP#4918)
  refactor(vault): Refactor vault package (SAP#5148)
  ...
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