Skip to content

Conversation

@gtcooke94
Copy link
Contributor

No description provided.

@markdroth markdroth changed the title L123: SPIFFE Bundle Map support in Providers L127: SPIFFE Bundle Map support in Providers Jul 16, 2025
@markdroth markdroth changed the title L127: SPIFFE Bundle Map support in Providers L127: C++: SPIFFE Bundle Map support in Providers Jul 16, 2025
@markdroth
Copy link
Member

Did you forget to push some of the changes? I don't see any but the first comment addressed.

@gtcooke94
Copy link
Contributor Author

My bad - editor confusion when moving the file from L123 to L127. Should be good now

@gtcooke94 gtcooke94 requested a review from markdroth July 17, 2025 14:48
This proposal _only_ aims to support file-based SPIFFE Bundle Maps via the file watcher providers. The `StaticDataCertificateProvider` structure is left as future work. This will involve broadening the API surface to expose a type for the SPIFFE bundle map.

## Implementation
PR will be linked when created.
Copy link
Member

Choose a reason for hiding this comment

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

I think you can list all of the PRs you've been working on that are directly related to this effort:

You can add to the list as you finish the remaining work.

copybara-service bot pushed a commit to grpc/grpc that referenced this pull request Aug 11, 2025
This PR adds APIs discussed in grpc/proposal#506 and https://github.com/grpc/proposal/blob/master/A87-mtls-spiffe-support.md to support verification with SPIFFE Bundle Map roots.

RELEASE NOTES:
* Adds support for SPIFFE Bundle Maps in as roots of trust per [gRFC A87](https://github.com/grpc/proposal/blob/master/A87-mtls-spiffe-support.md) and [gRFC L127](grpc/proposal#506)

Closes #40321

COPYBARA_INTEGRATE_REVIEW=#40321 from gtcooke94:spiffe_verification 6ddc24e
PiperOrigin-RevId: 793688101
@gtcooke94 gtcooke94 merged commit 343b4d8 into grpc:master Aug 11, 2025
1 check passed
eugeneo pushed a commit to eugeneo/grpc that referenced this pull request Aug 11, 2025
)

This PR adds APIs discussed in grpc/proposal#506 and https://github.com/grpc/proposal/blob/master/A87-mtls-spiffe-support.md to support verification with SPIFFE Bundle Map roots.

RELEASE NOTES:
* Adds support for SPIFFE Bundle Maps in as roots of trust per [gRFC A87](https://github.com/grpc/proposal/blob/master/A87-mtls-spiffe-support.md) and [gRFC L127](grpc/proposal#506)

Closes grpc#40321

COPYBARA_INTEGRATE_REVIEW=grpc#40321 from gtcooke94:spiffe_verification 6ddc24e
PiperOrigin-RevId: 793688101
gtcooke94 added a commit to grpc/grpc that referenced this pull request Aug 21, 2025
Cherry pick #40476 to 1.75

RELEASE NOTES:
* Adds support for SPIFFE Bundle Maps in as roots of trust per [gRFC
A87](https://github.com/grpc/proposal/blob/master/A87-mtls-spiffe-support.md)
and grpc/proposal#506
paulosjca pushed a commit to paulosjca/grpc that referenced this pull request Aug 23, 2025
)

This PR adds APIs discussed in grpc/proposal#506 and https://github.com/grpc/proposal/blob/master/A87-mtls-spiffe-support.md to support verification with SPIFFE Bundle Map roots.

RELEASE NOTES:
* Adds support for SPIFFE Bundle Maps in as roots of trust per [gRFC A87](https://github.com/grpc/proposal/blob/master/A87-mtls-spiffe-support.md) and [gRFC L127](grpc/proposal#506)

Closes grpc#40321

COPYBARA_INTEGRATE_REVIEW=grpc#40321 from gtcooke94:spiffe_verification 6ddc24e
PiperOrigin-RevId: 793688101
asheshvidyut pushed a commit to asheshvidyut/grpc that referenced this pull request Sep 12, 2025
)

This PR adds APIs discussed in grpc/proposal#506 and https://github.com/grpc/proposal/blob/master/A87-mtls-spiffe-support.md to support verification with SPIFFE Bundle Map roots.

RELEASE NOTES:
* Adds support for SPIFFE Bundle Maps in as roots of trust per [gRFC A87](https://github.com/grpc/proposal/blob/master/A87-mtls-spiffe-support.md) and [gRFC L127](grpc/proposal#506)

Closes grpc#40321

COPYBARA_INTEGRATE_REVIEW=grpc#40321 from gtcooke94:spiffe_verification 6ddc24e
PiperOrigin-RevId: 793688101
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