Skip to content

Comments

Add method to stream authority updates#59

Merged
sorindumitru merged 1 commit intospiffe:nextfrom
sorindumitru:get-trust-bundle
Jun 6, 2025
Merged

Add method to stream authority updates#59
sorindumitru merged 1 commit intospiffe:nextfrom
sorindumitru:get-trust-bundle

Conversation

@sorindumitru
Copy link
Contributor

@sorindumitru sorindumitru commented May 27, 2025

Currently the UpstreamAuthority protocol does not provide a way for us to listen to bundle updates without minting a new CA or publishing a new JWT key. This is problematic since it means that spire-server won't start listening for changes to upstream authorities until it first prepares a new key. If spire-server restarts and it has a key available there will be a period of time, up to the first key preparation, where it doesn't see any changes to the trust bundle of the trust domain.

Fix this by adding an optional method that can be used to only listen for trust domain updates.

part of the fix for spiffe/spire#6083

@sorindumitru sorindumitru force-pushed the get-trust-bundle branch 5 times, most recently from fbfd253 to d49bef2 Compare May 29, 2025 20:20
Signed-off-by: Sorin Dumitru <sdumitru@bloomberg.net>
@sorindumitru sorindumitru merged commit 68609d8 into spiffe:next Jun 6, 2025
4 checks passed
amartinezfayo pushed a commit to amartinezfayo/spire-plugin-sdk that referenced this pull request Jun 26, 2025
Signed-off-by: Sorin Dumitru <sdumitru@bloomberg.net>
amartinezfayo pushed a commit that referenced this pull request Jun 26, 2025
Signed-off-by: Sorin Dumitru <sdumitru@bloomberg.net>
Signed-off-by: Agustín Martínez Fayó <amartinezfayo@gmail.com>
amartinezfayo added a commit that referenced this pull request Jul 1, 2025
Signed-off-by: Sorin Dumitru <sdumitru@bloomberg.net>
Signed-off-by: Agustín Martínez Fayó <amartinezfayo@gmail.com>
Co-authored-by: Sorin Dumitru <sorin@returnze.ro>
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