You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Signing the contributor's agreement does not grant anyone commit rights to the main
44
-
repository, but it does mean that we can accept your contributions, and you will get an
45
-
author credit if we do. Active contributors might be asked to join the core team, and
46
-
given the ability to merge pull requests.
47
-
48
-
[[code-of-conduct]]
49
-
== Code of Conduct
50
-
This project adheres to the Contributor Covenant https://github.com/spring-cloud/spring-cloud-build/blob/main/docs/src/main/asciidoc/code-of-conduct.adoc[code of
51
-
conduct]. By participating, you are expected to uphold this code. Please report
* Make sure all new `.java` files to have a simple Javadoc class comment with at least an
67
-
`@author` tag identifying you, and preferably at least a paragraph on what the class is
68
-
for.
69
-
* Add the ASF license header comment to all new `.java` files (copy from existing files
70
-
in the project)
71
-
* Add yourself as an `@author` to the .java files that you modify substantially (more
72
-
than cosmetic changes).
73
-
* Add some Javadocs and, if you change the namespace, some XSD doc elements.
74
-
* A few unit tests would help a lot as well -- someone has to do it.
75
-
* If no-one else is using your branch, please rebase it against the current main (or
76
-
other target branch in the main project).
77
-
* When writing a commit message please follow https://tbaggery.com/2008/04/19/a-note-about-git-commit-messages.html[these conventions],
78
-
if you are fixing an existing issue please add `Fixes gh-XXXX` at the end of the commit
79
-
message (where XXXX is the issue number).
80
-
81
-
[[checkstyle]]
82
-
== Checkstyle
83
-
84
-
Spring Cloud Build comes with a set of checkstyle rules. You can find them in the `spring-cloud-build-tools` module. The most notable files under the module are:
85
-
86
-
.spring-cloud-build-tools/
87
-
----
88
-
└── src
89
-
├── checkstyle
90
-
│ └── checkstyle-suppressions.xml <3>
91
-
└── main
92
-
└── resources
93
-
├── checkstyle-header.txt <2>
94
-
└── checkstyle.xml <1>
95
-
----
96
-
<1> Default Checkstyle rules
97
-
<2> File header setup
98
-
<3> Default suppression rules
99
-
100
-
[[checkstyle-configuration]]
101
-
=== Checkstyle configuration
102
-
103
-
Checkstyle rules are *disabled by default*. To add checkstyle to your project just define the following properties and plugins.
<4> Add the Spring Java Format plugin that will reformat your code to pass most of the Checkstyle formatting rules
141
-
<5> Add checkstyle plugin to your build and reporting phases
142
-
143
-
If you need to suppress some rules (e.g. line length needs to be longer), then it's enough for you to define a file under `${project.root}/src/checkstyle/checkstyle-suppressions.xml` with your suppressions. Example:
It's advisable to copy the `${spring-cloud-build.rootFolder}/.editorconfig` and `${spring-cloud-build.rootFolder}/.springformat` to your project. That way, some default formatting rules will be applied. You can do so by running this script:
In order to setup Intellij you should import our coding conventions, inspection profiles and set up the checkstyle plugin.
171
-
The following files can be found in the https://github.com/spring-cloud/spring-cloud-build/tree/main/spring-cloud-build-tools[Spring Cloud Build] project.
172
-
173
-
.spring-cloud-build-tools/
174
-
----
175
-
└── src
176
-
├── checkstyle
177
-
│ └── checkstyle-suppressions.xml <3>
178
-
└── main
179
-
└── resources
180
-
├── checkstyle-header.txt <2>
181
-
├── checkstyle.xml <1>
182
-
└── intellij
183
-
├── Intellij_Project_Defaults.xml <4>
184
-
└── Intellij_Spring_Boot_Java_Conventions.xml <5>
185
-
----
186
-
<1> Default Checkstyle rules
187
-
<2> File header setup
188
-
<3> Default suppression rules
189
-
<4> Project defaults for Intellij that apply most of Checkstyle rules
190
-
<5> Project style conventions for Intellij that apply most of Checkstyle rules
Go to `File` -> `Settings` -> `Editor` -> `Code style`. There click on the icon next to the `Scheme` section. There, click on the `Import Scheme` value and pick the `Intellij IDEA code style XML` option. Import the `spring-cloud-build-tools/src/main/resources/intellij/Intellij_Spring_Boot_Java_Conventions.xml` file.
Go to `File` -> `Settings` -> `Editor` -> `Inspections`. There click on the icon next to the `Profile` section. There, click on the `Import Profile` and import the `spring-cloud-build-tools/src/main/resources/intellij/Intellij_Project_Defaults.xml` file.
203
-
204
-
.Checkstyle
205
-
206
-
To have Intellij work with Checkstyle, you have to install the `Checkstyle` plugin. It's advisable to also install the `Assertions2Assertj` to automatically convert the JUnit assertions
Go to `File` -> `Settings` -> `Other settings` -> `Checkstyle`. There click on the `+` icon in the `Configuration file` section. There, you'll have to define where the checkstyle rules should be picked from. In the image above, we've picked the rules from the cloned Spring Cloud Build repository. However, you can point to the Spring Cloud Build's GitHub repository (e.g. for the `checkstyle.xml` : `https://raw.githubusercontent.com/spring-cloud/spring-cloud-build/main/spring-cloud-build-tools/src/main/resources/checkstyle.xml`). We need to provide the following variables:
211
-
212
-
- `checkstyle.header.file` - please point it to the Spring Cloud Build's, `spring-cloud-build-tools/src/main/resources/checkstyle-header.txt` file either in your cloned repo or via the `https://raw.githubusercontent.com/spring-cloud/spring-cloud-build/main/spring-cloud-build-tools/src/main/resources/checkstyle-header.txt` URL.
213
-
- `checkstyle.suppressions.file` - default suppressions. Please point it to the Spring Cloud Build's, `spring-cloud-build-tools/src/checkstyle/checkstyle-suppressions.xml` file either in your cloned repo or via the `https://raw.githubusercontent.com/spring-cloud/spring-cloud-build/main/spring-cloud-build-tools/src/checkstyle/checkstyle-suppressions.xml` URL.
214
-
- `checkstyle.additional.suppressions.file` - this variable corresponds to suppressions in your local project. E.g. you're working on `spring-cloud-contract`. Then point to the `project-root/src/checkstyle/checkstyle-suppressions.xml` folder. Example for `spring-cloud-contract` would be: `/home/username/spring-cloud-contract/src/checkstyle/checkstyle-suppressions.xml`.
215
-
216
-
IMPORTANT: Remember to set the `Scan Scope` to `All sources` since we apply checkstyle rules for production and test sources.
217
-
218
-
[[duplicate-finder]]
219
-
== Duplicate Finder
220
-
221
-
Spring Cloud Build brings along the `basepom:duplicate-finder-maven-plugin`, that enables flagging duplicate and conflicting classes and resources on the java classpath.
222
-
223
-
[[duplicate-finder-configuration]]
224
-
=== Duplicate Finder configuration
225
-
226
-
Duplicate finder is *enabled by default* and will run in the `verify` phase of your Maven build, but it will only take effect in your project if you add the `duplicate-finder-maven-plugin` to the `build` section of the projecst's `pom.xml`.
For other properties, we have set defaults as listed in the https://github.com/basepom/duplicate-finder-maven-plugin/wiki[plugin documentation].
242
-
243
-
You can easily override them but setting the value of the selected property prefixed with `duplicate-finder-maven-plugin`. For example, set `duplicate-finder-maven-plugin.skip` to `true` in order to skip duplicates check in your build.
244
-
245
-
If you need to add `ignoredClassPatterns` or `ignoredResourcePatterns` to your setup, make sure to add them in the plugin configuration section of your project:
Signing the contributor's agreement does not grant anyone commit rights to the main
288
-
repository, but it does mean that we can accept your contributions, and you will get an
289
-
author credit if we do. Active contributors might be asked to join the core team, and
290
-
given the ability to merge pull requests.
46
+
All commits must include a __Signed-off-by__ trailer at the end of each commit message to indicate that the contributor agrees to the Developer Certificate of Origin.
47
+
For additional details, please refer to the blog post https://spring.io/blog/2025/01/06/hello-dco-goodbye-cla-simplifying-contributions-to-spring[Hello DCO, Goodbye CLA: Simplifying Contributions to Spring].
291
48
292
49
[[code-of-conduct]]
293
50
== Code of Conduct
294
-
This project adheres to the Contributor Covenant https://github.com/spring-cloud/spring-cloud-build/blob/main/docs/src/main/asciidoc/code-of-conduct.adoc[code of
51
+
This project adheres to the Contributor Covenant https://github.com/spring-cloud/spring-cloud-build/blob/main/docs/modules/ROOT/partials/code-of-conduct.adoc[code of
295
52
conduct]. By participating, you are expected to uphold this code. Please report
296
-
unacceptable behavior to spring-code-of-conduct@pivotal.io.
53
+
unacceptable behavior to code-of-conduct@spring.io.
297
54
298
55
[[code-conventions-and-housekeeping]]
299
56
== Code Conventions and Housekeeping
@@ -467,7 +224,7 @@ Spring Cloud Build brings along the `basepom:duplicate-finder-maven-plugin`, th
467
224
[[duplicate-finder-configuration]]
468
225
=== Duplicate Finder configuration
469
226
470
-
Duplicate finder is *enabled by default* and will run in the `verify` phase of your Maven build, but it will only take effect in your project if you add the `duplicate-finder-maven-plugin` to the `build` section of the projecst's `pom.xml`.
227
+
Duplicate finder is *enabled by default* and will run in the `verify` phase of your Maven build, but it will only take effect in your project if you add the `duplicate-finder-maven-plugin` to the `build` section of the project's `pom.xml`.
Copy file name to clipboardExpand all lines: docs/modules/ROOT/pages/spring-cloud-gateway-server-mvc/filters/loadbalancer.adoc
+2Lines changed: 2 additions & 0 deletions
Display the source diff
Display the rich diff
Original file line number
Diff line number
Diff line change
@@ -43,6 +43,8 @@ spring:
43
43
- Path=/api/**
44
44
----
45
45
46
+
WARNING: If using the `lb()` filter, it needs to be after any filter that manipulates the path such as `setPath()` or `stripPrefix()`, otherwise the resulting url could be incorrect. The `lb:` scheme handler in configuration, automatically puts the filter in the highest precedence order.
47
+
46
48
NOTE: By default, when a service instance cannot be found by the `ReactorLoadBalancer`, a `503` is returned.
47
49
// TODO: implement use404
48
50
// You can configure the gateway to return a `404` by setting `spring.cloud.gateway.loadbalancer.use404=true`.
Copy file name to clipboardExpand all lines: docs/modules/ROOT/pages/spring-cloud-gateway-server-mvc/filters/prefixpath.adoc
+2-1Lines changed: 2 additions & 1 deletion
Display the source diff
Display the rich diff
Original file line number
Diff line number
Diff line change
@@ -33,7 +33,7 @@ class RouteConfiguration {
33
33
public RouterFunction<ServerResponse> gatewayRouterFunctionsPrefixPath() {
34
34
return route("prefixpath_route")
35
35
.GET("/**", http("https://example.org"))
36
-
.before("/mypath")
36
+
.before(prefixPath("/mypath"))
37
37
.build();
38
38
}
39
39
}
@@ -42,3 +42,4 @@ class RouteConfiguration {
42
42
This prefixes `/mypath` to the path of all matching requests.
43
43
So a request to `/hello` is sent to `/mypath/hello`.
44
44
45
+
WARNING: If using the `lb()` filter, it needs to be after the `prefixPath()` filter, otherwise the resulting url could be incorrect. The `lb:` scheme handler in configuration, automatically puts the filter in the highest precedence order.
Copy file name to clipboardExpand all lines: docs/modules/ROOT/pages/spring-cloud-gateway-server-mvc/filters/retry.adoc
+9-1Lines changed: 9 additions & 1 deletion
Display the source diff
Display the rich diff
Original file line number
Diff line number
Diff line change
@@ -9,6 +9,7 @@ The `Retry` filter supports the following parameters:
9
9
* `methods`: The HTTP methods that should be retried, represented by using `org.springframework.http.HttpMethod`.
10
10
* `series`: The series of status codes to be retried, represented by using `org.springframework.http.HttpStatus.Series`.
11
11
* `exceptions`: A list of thrown exceptions that should be retried.
12
+
* `cacheBody`: A flag to signal if the request body should be cached. If set to `true`, the `adaptCacheBody` filter must be used to send the cached body downstream.
12
13
//* `backoff`: The configured exponential backoff for the retries.
13
14
//Retries are performed after a backoff interval of `firstBackoff * (factor ^ n)`, where `n` is the iteration.
14
15
//If `maxBackoff` is configured, the maximum backoff applied is limited to `maxBackoff`.
@@ -20,8 +21,11 @@ The following defaults are configured for `Retry` filter, if enabled:
20
21
* `series`: 5XX series
21
22
* `methods`: GET method
22
23
* `exceptions`: `IOException`, `TimeoutException` and `RetryException`
24
+
* `cacheBody`: `false`
23
25
//* `backoff`: disabled
24
26
27
+
WARNING: Setting `cacheBody` to `true` causes the gateway to read the whole body into memory. This should be used with caution.
0 commit comments