jenkinsci / jenkinsci/github-branch-source-plugin
[JENKINS-73787] Revisit snapshotting of GitHubAppCredentials
- Dominant language
- Java
- Stars
- 217
- Forks
- 398
- Avg merge
- 30m
- Merged PRs (30d)
- 1
Description
The current design of the GitHubAppCredentials snapshotting is messy as discussed in https://github.com/jenkinsci/github-branch-source-plugin/pull/810#discussion_r1761493599.
GitHubAppCredentials#writeReplace directly return a DelegatingGitHubAppCredentials which should not be necessary, or rather should rely on the CredentialsSnapshotTaker entirely.
---
Originally reported by
allan_burdajewicz, imported from: Revisit snapshotting of GitHubAppCredentials
Raw content of original issue
The current design of the GitHubAppCredentials snapshotting is messy as discussed in https://github.com/jenkinsci/github-branch-source-plugin/pull/810#discussion_r1761493599.
GitHubAppCredentials#writeReplace directly return a DelegatingGitHubAppCredentials which should not be necessary, or rather should rely on the CredentialsSnapshotTaker entirely.
Contributor guide
Assessment
This issue has not been assessed yet.