Approvals: 0/1
[08:57:21] * ChanServ sets mode: +o temporalfox [09:08:15] * ChanServ sets mode: +o temporal_
[09:46:53] <aesteve> hi everyone
[09:47:43] <aesteve> pmlopes: I've seen the issue re. path params / query params and I was thinking of a design that'd not break current implementation
[09:49:31] <aesteve> I think we could keep things as is (i.e. still doing request.params().addAll(pathParams)), but moreover doing routingContext.contextData().putAll(“$$vertx-routing-params$$”, pathParams);
[09:50:20] <aesteve> thus it'd be backward compatible, and fixing the issue if people need an actual route param (related to vertx-web and not vertx standard)
[09:50:24] <aesteve> wdyt ?
[10:56:17] <aesteve> ok pmlopes I submitted a PR, let me know what you think about it, at least we have a basic implementation to start discussing
[11:52:20] * ChanServ sets mode: +o temporalfox
[11:54:31] <aesteve> hi temporalfox :) how are you ?
[11:54:45] <temporalfox> hi better than the past two days that were abysmal for me
[11:55:33] <aesteve> you looked fine though !
[11:56:05] <temporalfox> it was horrible
[11:56:17] <temporalfox> I was fine on wed
[11:56:30] <temporalfox> terrible headaches on thursday and yesterday
[11:57:12] <aesteve> aw sounds bad indeed
[12:05:32] <aesteve> I submitted a PR but it's for vertx-web, can you point me at the PR re. the futures composition ? so that I can read it
[12:13:37] <temporalfox> ah
[12:13:43] <temporalfox> hum it's only local code for now
[12:13:51] <temporalfox> there is some already in core
[12:14:05] <temporalfox> with compose that use a Function and map
[12:14:26] <temporalfox> https://github.com/eclipse/vert.x/commit/f1be65f13b3f13032494904d4444620d712a73cd
[12:14:34] <temporalfox> and have still a couple of extra commits
[12:16:25] <temporalfox> but it does not add that much
[12:16:30] <temporalfox> I can point them to you later
[12:16:32] <aesteve> thx I'll have a look
[12:16:37] <temporalfox> I'm going back to marseille this afternoon
[12:18:46] <aesteve> ok
[12:19:03] <aesteve> ok this looks good to me
[12:20:37] <aesteve> did you add something like that, too https://github.com/aesteve/nubes/blob/e03e611418d50364cd3fc0e702e3bc49f9298c6b/src/main/java/com/github/aesteve/vertx/nubes/utils/async/AsyncUtils.java#L120
[12:20:41] <aesteve> kind of an iterator
[12:21:01] <aesteve> (I agree I should have used RxJava for this kind of stuff)
[12:22:19] <temporalfox> will ahve a look in th etrain
[12:22:23] <temporalfox> going soon to the station
[12:22:33] <aesteve> ok, have a nice trip !
[12:22:53] <aesteve> basically I have put everything I kinda needed quite often in AsyncUtils
[12:23:20] <aesteve> so if you're looking at real-life use cases this is where I put every one of mines
[12:23:32] <aesteve> looking for*
[14:23:41] <aesteve> hi pmlopes do you have a few minutes ? so that we can talk about the PR ?
[14:23:52] <aesteve> (especially the reroute() behaviour)
[14:24:59] * ChanServ sets mode: +o temporalfox
[14:29:31] * ChanServ sets mode: +o temporal_ [14:43:29] * ChanServ sets mode: +o temporalfox
[15:34:10] * ChanServ sets mode: +o temporal_ [17:20:06] * ChanServ sets mode: +o temporalfox
[18:34:54] *** ChanServ sets mode: +o temporalfox
[20:21:38] <AlexLehm> i have a junit question, for the https proxy test I have created a new test class to be able to additional start and stop calls (@Before and @After)
[20:22:07] <AlexLehm> would it be better to do that in the normal https test class and use try/finaly instead or use custom rules
[20:22:40] <AlexLehm> or it would possible to use a BeforeClass / AfterClass method to start the proxy only once