Skip to content

Support nested builder in DSL for reactive apps#7121

Merged
rwinch merged 1 commit intospring-projects:masterfrom
eleftherias:gh-7107-server-dsl-nested-builder
Jul 23, 2019
Merged

Support nested builder in DSL for reactive apps#7121
rwinch merged 1 commit intospring-projects:masterfrom
eleftherias:gh-7107-server-dsl-nested-builder

Conversation

@eleftherias
Copy link
Copy Markdown
Contributor

Fixes: gh-7107

@eleftherias eleftherias force-pushed the gh-7107-server-dsl-nested-builder branch from 22a68d0 to 4af9c30 Compare July 23, 2019 12:45
Copy link
Copy Markdown
Member

@rwinch rwinch left a comment

Choose a reason for hiding this comment

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

Thanks for the PR! I left feedback inline

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

To be consistent with the variable we typically use, please change to http. We also use http to make it look similar to the XML based configuration.

@eleftherias eleftherias force-pushed the gh-7107-server-dsl-nested-builder branch from 4af9c30 to a0f4fff Compare July 23, 2019 16:28
@rwinch rwinch assigned rwinch and unassigned rwinch Jul 23, 2019
@rwinch rwinch added in: config An issue in spring-security-config status: duplicate A duplicate of another issue type: enhancement A general enhancement labels Jul 23, 2019
@rwinch rwinch added this to the 5.2.0.M4 milestone Jul 23, 2019
@rwinch rwinch merged commit a288ce4 into spring-projects:master Jul 23, 2019
@rwinch
Copy link
Copy Markdown
Member

rwinch commented Jul 23, 2019

Thanks this is now merged into master!

@eleftherias eleftherias deleted the gh-7107-server-dsl-nested-builder branch August 26, 2019 15:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

in: config An issue in spring-security-config status: duplicate A duplicate of another issue type: enhancement A general enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add support for nested builders in the DSL for reactive apps

2 participants