Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Possible NPE in CachingSpringLoadBalancerFactory #5

Closed
mcoulombe636 opened this issue Mar 6, 2018 · 0 comments
Closed

Possible NPE in CachingSpringLoadBalancerFactory #5

mcoulombe636 opened this issue Mar 6, 2018 · 0 comments
Milestone
2.0.0.M2

Comments

@mcoulombe636
Copy link
Contributor

Bug report

As discussed here a race condition with the GC can cause an NPE in the CachingSpringLoadBalancerFactory
mcoulombe636 added a commit to mcoulombe636/spring-cloud-openfeign that referenced this issue Mar 6, 2018
@mcoulombe636
Fix the race condition with the GC since the ConcurrentReferenceHashM…
fba4651 
…ap holds soft references to clients.

Fixes spring-cloudgh-5
spring-cloud#5
@mcoulombe636 mcoulombe636 mentioned this issue Mar 6, 2018
Merged
ryanjbaxter pushed a commit that referenced this issue Mar 9, 2018
@mcoulombe636 @ryanjbaxter
Fix the race condition with the GC since the ConcurrentReferenceHashM…
c45f153 
…ap holds soft references to clients. (#6)

Fixes gh-5
@ryanjbaxter ryanjbaxter added this to the 2.0.0.RC1 milestone Mar 9, 2018
holy12345 added a commit to holy12345/spring-cloud-openfeign that referenced this issue Apr 29, 2018
@holy12345
Pull the code from master branch (#1)
ae9a491 
* Update SNAPSHOT to 2.0.0.M1

* Going back to snapshots

* codecov.yml for comments on pr

* Added codecov link to readme

* Adding codacy badge

* Fix the race condition with the GC since the ConcurrentReferenceHashMap holds soft references to clients. (spring-cloud#6)

Fixes spring-cloudgh-5

* Refactor spring retry code (spring-cloud#10)

* Fix NPE when body is empty.  Fixes spring-cloud#12.

* FIxing links

* Update SNAPSHOT to 2.0.0.M2

* Going back to snapshots

* Update SNAPSHOT to 2.0.0.RC1

* Going back to snapshots
@stela stela mentioned this issue Dec 27, 2019
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
2.0.0.M2
Development

No branches or pull requests
2 participants
@ryanjbaxter @mcoulombe636