Skip to content

Middleware configuration, extension, HTTP and gRPC support all in one #12842

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

Closed
wants to merge 5 commits into from

Conversation

jmacd
Copy link
Contributor

@jmacd jmacd commented Apr 15, 2025

Description

This is the first major portion of #12700 (see #12603, #9591, #7441). It includes configmiddleware, extensionmiddleware, and changes in confighttp and configgrpc all in one.

Link to tracking issue

Part of #12603.

Testing

New tests are added in each config/* package. The extension API is tested from the configmiddleware package.

Documentation

Added.

github-merge-queue bot pushed a commit that referenced this pull request Apr 15, 2025
#### Description

Adds the extension API from #12842.

#### Link to tracking issue

Part of #12603.

#### Testing

Not tested. See configmiddleware.

#### Documentation

Added.
github-merge-queue bot pushed a commit that referenced this pull request Apr 17, 2025
**Description**

Adds the config struct from
#12842.

**Link to tracking issue**

Part of
#12603.

**Testing**

Yes. This PR introduces `extensionmiddlewaretest` helpers.

**Documentation**

Added.

---------

Co-authored-by: Bogdan Drutu <[email protected]>
@jmacd
Copy link
Contributor Author

jmacd commented Apr 18, 2025

In favor of #12845 #12846 #12843 #12844

@jmacd jmacd closed this Apr 18, 2025
github-merge-queue bot pushed a commit that referenced this pull request Apr 18, 2025
#### Description

Adds the HTTP middleware support from #12842.

#### Link to tracking issue

Part of #12603.

#### Testing

Yes.

#### Documentation

Added.
github-merge-queue bot pushed a commit that referenced this pull request Apr 19, 2025
#### Description

Adds the gRPC middleware support from #12842.

#### Link to tracking issue

Part of #12603.

#### Testing

Yes.

#### Documentation

Added.
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.

1 participant