-
Notifications
You must be signed in to change notification settings - Fork 566
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
Prototype builder update #7281
Prototype builder update #7281
Conversation
- builder no longer implements prototype - interceptor changed to decorator
- builder methods now return Optional where appropriate - builder interceptor now a decorator
1a5c15a
to
633b19d
Compare
Rebased on main |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The following does not compile anymore.
WebServer.builder().mediaContext(MediaContext.builder());
The builder should have a variant of the mediaContext
method that accepts Supplier<MediaContext>
.
nima/webclient/api/src/main/java/io/helidon/nima/webclient/api/HttpClientConfigBlueprint.java
Show resolved
Hide resolved
...ebclient/http1/src/main/java/io/helidon/nima/webclient/http1/Http1ClientConfigBlueprint.java
Show resolved
Hide resolved
I would prefer to add the methods that accept supplier as a follow up to this issue. This is fixing the problem of sending builder where we should have sent a built value. |
Fixes #7212
Fixes #7283
Also changed interceptor to decorator, as interceptor not needed for builders