Ryan Baxter
ab1f1f43a8
Fixing tests
6 years ago
Ryan Baxter
c51117d556
Merge branch 'polish-urls-remaining-2.0.x' of https://github.com/spring-operator/spring-cloud-openfeign into spring-operator-polish-urls-remaining-2.0.x
6 years ago
Spring Operator
d6ad74a933
URL Cleanup ( #162 )
...
This commit updates URLs to prefer the https protocol. Redirects are not followed to avoid accidentally expanding intentionally shortened URLs (i.e. if using a URL shortener).
# Fixed URLs
## Fixed Success
These URLs were switched to an https URL with a 2xx status. While the status was successful, your review is still recommended.
* [ ] http://repo.spring.io/libs-milestone-local with 2 occurrences migrated to:
https://repo.spring.io/libs-milestone-local ([https](https://repo.spring.io/libs-milestone-local ) result 302).
* [ ] http://repo.spring.io/libs-snapshot-local with 2 occurrences migrated to:
https://repo.spring.io/libs-snapshot-local ([https](https://repo.spring.io/libs-snapshot-local ) result 302).
* [ ] http://repo.spring.io/release with 1 occurrences migrated to:
https://repo.spring.io/release ([https](https://repo.spring.io/release ) result 302).
# Ignored
These URLs were intentionally ignored.
* http://maven.apache.org/POM/4.0.0 with 10 occurrences
* http://www.w3.org/2001/XMLSchema-instance with 5 occurrences
6 years ago
Spring Operator
6aa6684d13
URL Cleanup
...
This commit updates URLs to prefer the https protocol. Redirects are not followed to avoid accidentally expanding intentionally shortened URLs (i.e. if using a URL shortener).
# Fixed URLs
## Fixed But Review Recommended
These URLs were fixed, but the https status was not OK. However, the https status was the same as the http request or http redirected to an https URL, so they were migrated. Your review is recommended.
* [ ] http://foo.com:8000/ (ConnectTimeoutException) with 2 occurrences migrated to:
https://foo.com:8000/ ([https](https://foo.com:8000/ ) result ConnectTimeoutException).
* [ ] http://foo.com:8000/abc/bcd.json (ConnectTimeoutException) with 1 occurrences migrated to:
https://foo.com:8000/abc/bcd.json ([https](https://foo.com:8000/abc/bcd.json ) result ConnectTimeoutException).
* [ ] http://compose.docker.io/ (UnknownHostException) with 1 occurrences migrated to:
https://compose.docker.io/ ([https](https://compose.docker.io/ ) result UnknownHostException).
* [ ] http://tp/abc/bcd.json (UnknownHostException) with 1 occurrences migrated to:
https://tp/abc/bcd.json ([https](https://tp/abc/bcd.json ) result UnknownHostException).
* [ ] http://www.puppycrawl.com/dtds/suppressions_1_1.dtd (404) with 1 occurrences migrated to:
https://www.puppycrawl.com/dtds/suppressions_1_1.dtd ([https](https://www.puppycrawl.com/dtds/suppressions_1_1.dtd ) result 404).
## Fixed Success
These URLs were switched to an https URL with a 2xx status. While the status was successful, your review is still recommended.
* [ ] http://asciidoctor.org with 1 occurrences migrated to:
https://asciidoctor.org ([https](https://asciidoctor.org ) result 200).
* [ ] http://example.com with 2 occurrences migrated to:
https://example.com ([https](https://example.com ) result 200).
* [ ] http://github.com/ with 1 occurrences migrated to:
https://github.com/ ([https](https://github.com/ ) result 200).
* [ ] http://projects.spring.io/spring-cloud/ with 1 occurrences migrated to:
https://projects.spring.io/spring-cloud/ ([https](https://projects.spring.io/spring-cloud/ ) result 200).
* [ ] http://tbaggery.com/2008/04/19/a-note-about-git-commit-messages.html with 2 occurrences migrated to:
https://tbaggery.com/2008/04/19/a-note-about-git-commit-messages.html ([https](https://tbaggery.com/2008/04/19/a-note-about-git-commit-messages.html ) result 200).
* [ ] http://plugins.jetbrains.com/plugin/6546 with 2 occurrences migrated to:
https://plugins.jetbrains.com/plugin/6546 ([https](https://plugins.jetbrains.com/plugin/6546 ) result 301).
* [ ] http://raw.github.com/ with 1 occurrences migrated to:
https://raw.github.com/ ([https](https://raw.github.com/ ) result 301).
* [ ] http://service.com with 1 occurrences migrated to:
https://service.com ([https](https://service.com ) result 301).
* [ ] http://eclipse.org with 1 occurrences migrated to:
https://eclipse.org ([https](https://eclipse.org ) result 302).
* [ ] http://eclipse.org/m2e/ with 2 occurrences migrated to:
https://eclipse.org/m2e/ ([https](https://eclipse.org/m2e/ ) result 302).
* [ ] http://www.springsource.com/developer/sts with 1 occurrences migrated to:
https://www.springsource.com/developer/sts ([https](https://www.springsource.com/developer/sts ) result 302).
# Ignored
These URLs were intentionally ignored.
* http://PROD-SVC with 2 occurrences
* http://Url/ with 1 occurrences
* http://bad_hostname with 1 occurrences
* http://bar with 1 occurrences
* http://bar/ with 1 occurrences
* http://foo with 18 occurrences
* http://foo/ with 5 occurrences
* http://foo/?name=%7bcookie with 1 occurrences
* http://foo:7777/ with 1 occurrences
* http://good-name with 2 occurrences
* http://listener with 6 occurrences
* http://localhost with 11 occurrences
* http://localhost:8080/ with 1 occurrences
* http://localhost:8081/ with 1 occurrences
6 years ago
Spring Operator
11098e9a99
URL Cleanup ( #155 )
...
This commit updates URLs to prefer the https protocol. Redirects are not followed to avoid accidentally expanding intentionally shortened URLs (i.e. if using a URL shortener).
# Fixed URLs
## Fixed Success
These URLs were switched to an https URL with a 2xx status. While the status was successful, your review is still recommended.
* [ ] http://www.apache.org/licenses/ with 1 occurrences migrated to:
https://www.apache.org/licenses/ ([https](https://www.apache.org/licenses/ ) result 200).
* [ ] http://www.apache.org/licenses/LICENSE-2.0 with 102 occurrences migrated to:
https://www.apache.org/licenses/LICENSE-2.0 ([https](https://www.apache.org/licenses/LICENSE-2.0 ) result 200).
6 years ago
Spring Operator
36cc521a81
URL Cleanup ( #148 )
...
This commit updates URLs to prefer the https protocol. Redirects are not followed to avoid accidentally expanding intentionally shortened URLs (i.e. if using a URL shortener).
# Fixed URLs
## Fixed Success
These URLs were switched to an https URL with a 2xx status. While the status was successful, your review is still recommended.
* http://maven.apache.org/xsd/maven-4.0.0.xsd with 5 occurrences migrated to:
https://maven.apache.org/xsd/maven-4.0.0.xsd ([https](https://maven.apache.org/xsd/maven-4.0.0.xsd ) result 200).
* http://stackoverflow.com/questions/1593051/how-to-programmatically-determine-the-current-checked-out-git-branch with 1 occurrences migrated to:
https://stackoverflow.com/questions/1593051/how-to-programmatically-determine-the-current-checked-out-git-branch ([https](https://stackoverflow.com/questions/1593051/how-to-programmatically-determine-the-current-checked-out-git-branch ) result 200).
* http://stackoverflow.com/questions/29300806/a-bash-script-to-check-if-a-string-is-present-in-a-comma-separated-list-of-strin with 1 occurrences migrated to:
https://stackoverflow.com/questions/29300806/a-bash-script-to-check-if-a-string-is-present-in-a-comma-separated-list-of-strin ([https](https://stackoverflow.com/questions/29300806/a-bash-script-to-check-if-a-string-is-present-in-a-comma-separated-list-of-strin ) result 200).
* http://www.apache.org/licenses/LICENSE-2.0 with 2 occurrences migrated to:
https://www.apache.org/licenses/LICENSE-2.0 ([https](https://www.apache.org/licenses/LICENSE-2.0 ) result 200).
# Ignored
These URLs were intentionally ignored.
* http://maven.apache.org/POM/4.0.0 with 10 occurrences
* http://www.w3.org/2001/XMLSchema-instance with 5 occurrences
6 years ago
buildmaster
3a79850467
Bumping versions to 2.0.4.BUILD-SNAPSHOT after release
6 years ago
buildmaster
117e2c2b01
Going back to snapshots
6 years ago
buildmaster
80de4e6b66
Update SNAPSHOT to 2.0.3.RELEASE
6 years ago
buildmaster
5f4acbe80e
Bumping versions
6 years ago
Ryan Baxter
a693c30930
Add null check for retry policy. Fixes #101
6 years ago
Ryan Baxter
4e7e187323
Fix bug where host contains protocol characters ( #97 )
...
Fixes #94
6 years ago
wzslw
88a6682f46
Update pom.xml ( #88 )
...
Remove duplicate dependency
6 years ago
buildmaster
5ddd771ed5
Bumping versions to 2.0.3.BUILD-SNAPSHOT after release
6 years ago
buildmaster
9f9c41ab59
Going back to snapshots
6 years ago
buildmaster
4dd68ca2f2
Update SNAPSHOT to 2.0.2.RELEASE
6 years ago
Ryan Baxter
af094d0bfc
Remove rogue content in docs
6 years ago
Ryan Baxter
77cdd40f7d
Add missing properties to additional-spring-configuration-metadata. Fixes #61
6 years ago
sdoeringNew
750cb35e6a
Add builder for feign clients. ( #56 )
...
Utilizes the logic from the registrar and the factory bean. The changes to them have been made minimal invasive.
Fixes gh-42
7 years ago
Kerwin Bryant
b512c451fd
optimized automatic configuration ( #70 )
7 years ago
Ryan Baxter
da6a51778f
Merge branch '2.0.x' of github.com:spring-cloud/spring-cloud-openfeign into 2.0.x
7 years ago
Ryan Baxter
9fc19bb6fe
Updating dependencies. Fixes #68
7 years ago
Eugene
e0bf63bed1
Added support for MultipartFile ( #66 )
...
Fixes gh-62
### Notes
The solution is based on
```
<dependency>
<groupId>io.github.openfeign.form</groupId>
<artifactId>feign-form-spring</artifactId>
<version>3.3.0</version>
</dependency>
```
Class `feign.form.spring.SpringFormEncoder` was added inside `org.springframework.cloud.openfeign.support.SpringEncoder` because it's doesn't has some checks:
1. Check of `bodyType` on null.
`org.springframework.cloud.openfeign.support.SpringEncoder#encode` don't handle situation when `bodyType` is null.
2. No check for content type `multipart/form-data` in header.
`feign.form.FormEncoder` requires `multipart/form-data` in header for proper processing.
All this checks added and tests are passed.
7 years ago
Nicklas Holm
a33fd6d869
Add spring boot configuration processor. Fixes #63 ( #64 )
7 years ago
Ryan Baxter
06ccb9d1e5
Remove code which was calling FallbackFactory with RuntimeException. Fixes #41 ( #58 )
7 years ago
Ryan Baxter
d57cea265e
Fix invalid URI error when Feign request URI does not end in /. See sc-netflix #3136
7 years ago
buildmaster
34cb91bc4c
Bumping versions to 2.0.2.BUILD-SNAPSHOT after release
7 years ago
buildmaster
58dc45d230
Going back to snapshots
7 years ago
buildmaster
80af701ccf
Update SNAPSHOT to 2.0.1.RELEASE
7 years ago
Ryan Baxter
46bc5da424
Make netflix starters optional. Fixes #35 ( #36 )
7 years ago
Ryan Baxter
7dcb696ec4
Use OK Http client bean instead of instance variable. Fixes #46
7 years ago
Ryan Baxter
69461d4385
Adding spring boot autoconfigure processor. Fixes #43 .
7 years ago
Matt Benson
8364b16b8e
expose configuration metadata for property 'feign.hystrix.enabled' ( #40 )
7 years ago
Ian Hu
6d4686624e
Fix typo in comments ( #38 )
7 years ago
buildmaster
51ad63231d
Bumping versions to 2.0.1.BUILD-SNAPSHOT after release
7 years ago
buildmaster
6c09501d77
Going back to snapshots
7 years ago
buildmaster
7e1c91c8b8
Update SNAPSHOT to 2.0.0.RELEASE
7 years ago
Spencer Gibb
674b88b467
Updates to s-c-build 2.0.2.BUILD-SNAPSHOT
7 years ago
Fernando F D Bittencourt
a57c416d4a
Explaining the return in case of exception. ( #32 )
7 years ago
Karl Nicholas
ffd4290e06
Update package to correct package in two test files. ( #30 )
7 years ago
Ryan Baxter
804ef91620
Merge branch 'master' of github.com:spring-cloud/spring-cloud-openfeign
7 years ago
Ryan Baxter
48e2f70757
Updating sc-build to 2.0.2.BUILD-SNAPSHOT
7 years ago
Rafael Renan Pacheco
569628689b
Format "Creating Feign Clients Manually" snippet ( #29 )
7 years ago
buildmaster
af02a8c692
Going back to snapshots
7 years ago
buildmaster
21be1e129c
Update SNAPSHOT to 2.0.0.RC2
7 years ago
Gerasimov Gennady
18111801d3
Misspelling fix according to https://github.com/spring-cloud/spring-cloud-openfeign/issues/24 ( #25 )
7 years ago
Ryan Baxter
e27d4d38e5
Fixing compatibility issues between netflix and openfeign as well as boot 1.5.x and 2.0.x
7 years ago
Ryan Baxter
01a2cda776
Remove misplaced StringEncoder setting when dealing with protobuf. Porting change from sc-netflix #2453
7 years ago
Ryan Baxter
9ac36c7f3f
Use type parameter instead of class. Fixes problem when OK HTTP is not on the classpath.
7 years ago
buildmaster
f4fc531857
Going back to snapshots
7 years ago