cescoffier / vertx-microservices-workshop

Vert.x Microservices Hand's on lab
http://escoffier.me/vertx-hol/
Apache License 2.0
330 stars 196 forks source link

Chain may loose exception #15

Open KrzysztofKowalczyk opened 7 years ago

KrzysztofKowalczyk commented 7 years ago

Have found Chain class in the vertx wishlist.

I think that every Function.apply should be wrapped in try / catch block as it may throw an exception which would not be propagated to returned future.

cescoffier commented 7 years ago

Definitely !