[03:46] *** Xliff joined
[04:57] *** Xliff left
[11:13] <Geth> ¦ cro-http/reverse-proxy: 9012ccca01 | Altai-man++ | lib/Cro/HTTP/ReverseProxy.pm6

[11:13] <Geth> ¦ cro-http/reverse-proxy: Tweaks

[11:13] <Geth> ¦ cro-http/reverse-proxy: review: https://github.com/croservices/cro-http/commit/9012ccca01

[11:50] *** Altai-man joined
[13:57] <Geth> ¦ cro-http/reverse-proxy: a1954aa4d9 | Altai-man++ | 3 files

[13:57] <Geth> ¦ cro-http/reverse-proxy: Tweaks

[13:57] <Geth> ¦ cro-http/reverse-proxy: review: https://github.com/croservices/cro-http/commit/a1954aa4d9

[13:57] <Geth> ¦ cro-http/reverse-proxy: 4bf013464c | Altai-man++ | lib/Cro/HTTP/ReverseProxy.pm6

[13:57] <Geth> ¦ cro-http/reverse-proxy: Remove await to utilize whenever and properly pass $*PROXY-STATE

[13:57] <Geth> ¦ cro-http/reverse-proxy: review: https://github.com/croservices/cro-http/commit/4bf013464c

[14:37] <Geth> ¦ cro-http/reverse-proxy: dbb7f9574b | Altai-man++ | 2 files

[14:37] <Geth> ¦ cro-http/reverse-proxy: Add a test for modifying request body

[14:37] <Geth> ¦ cro-http/reverse-proxy: review: https://github.com/croservices/cro-http/commit/dbb7f9574b

[14:49] <Geth> ¦ cro-http/reverse-proxy: 3115fd4ac8 | Altai-man++ | t/http-reverse-proxy.t

[14:49] <Geth> ¦ cro-http/reverse-proxy: Use a more suitable HTTP method

[14:49] <Geth> ¦ cro-http/reverse-proxy: review: https://github.com/croservices/cro-http/commit/3115fd4ac8

[14:53] <Geth> ¦ cro-http: 934ae8896e | Altai-man++ | t/http-server.t

[14:53] <Geth> ¦ cro-http: Suppress error output during a test run

[14:53] <Geth> ¦ cro-http: 

[14:53] <Geth> ¦ cro-http: The test covers an error condition, and Cro dumps errors to $*ERR by

[14:53] <Geth> ¦ cro-http: default. It's entirely normal behavior, but can make it look like there

[14:53] <Geth> ¦ cro-http: was some kind of failure during testing, so suppress that output.

[14:53] <Geth> ¦ cro-http: review: https://github.com/croservices/cro-http/commit/934ae8896e

[14:59] <Geth> ¦ cro-http/master: 21 commits pushed by Altai-man++, (Jonathan Worthington)++

[14:59] <Geth> ¦ cro-http/master: review: https://github.com/croservices/cro-http/compare/934ae8896e70...f37221f21912

[15:01] <Geth> ¦ cro: 5750acde82 | (Jonathan Worthington)++ | docs/index.md

[15:01] <Geth> ¦ cro: Add link to reverse proxy docs

[15:01] <Geth> ¦ cro: 

[15:01] <Geth> ¦ cro: Now that the features has been merged.

[15:01] <Geth> ¦ cro: review: https://github.com/croservices/cro/commit/5750acde82

[15:11] <Geth> ¦ cro-webapp: f88ef6fe9a | (Jonathan Worthington)++ | 7 files

[15:11] <Geth> ¦ cro-webapp: Provide for iteration with a separator

[15:11] <Geth> ¦ cro-webapp: 

[15:11] <Geth> ¦ cro-webapp: Resolves #47, which requested a way to do an iteration with a separator

[15:11] <Geth> ¦ cro-webapp: that goes between the items. Given this is a relatively common need and

[15:11] <Geth> ¦ cro-webapp: there's no easy way to do it otherwise, we add this way to do it.

[15:11] <Geth> ¦ cro-webapp: review: https://github.com/croservices/cro-webapp/commit/f88ef6fe9a

[15:11] <Geth> ¦ cro-webapp: 404e8eef3e | (Jonathan Worthington)++ (committed using GitHub Web editor) | 7 files

[15:11] <Geth> ¦ cro-webapp: Merge pull request #60 from croservices/separators

[15:11] <Geth> ¦ cro-webapp: 

[15:11] <Geth> ¦ cro-webapp: Provide for iteration with a separator

[15:11] <Geth> ¦ cro-webapp: review: https://github.com/croservices/cro-webapp/commit/404e8eef3e

[15:11] <Geth> ¦ cro: f732cf8bce | (Jonathan Worthington)++ | docs/reference/cro-webapp-template.md

[15:11] <Geth> ¦ cro: Document Cro template iteration separators

[15:12] <Geth> ¦ cro: review: https://github.com/croservices/cro/commit/f732cf8bce

[15:16] <Geth> ¦ cro: 252d40156e | (Jonathan Worthington)++ | docs/reference/cro-webapp-form.md

[15:16] <Geth> ¦ cro: Document new Cro form feature to set HTTP method

[15:16] <Geth> ¦ cro: review: https://github.com/croservices/cro/commit/252d40156e

[15:22] <Geth> ¦ cro-http/link-gen-no-fork: 8 commits pushed by vendethiel++, Altai-man++

[15:22] <Geth> ¦ cro-http/link-gen-no-fork: 6f68c78b30 | Link generation from route definition

[15:22] <Geth> ¦ cro-http/link-gen-no-fork: e014103ce1 | Update META6.json with new files

[15:22] <Geth> ¦ cro-http/link-gen-no-fork: 278bd0c42e | Streamline route naming story

[15:22] <Geth> ¦ cro-http/link-gen-no-fork: 28f7af1010 | Fix construction of URLs, properly resolve names in inner blocks

[15:22] <Geth> ¦ cro-http/link-gen-no-fork: 9274dd53f5 | Remove obsolete rewritten names

[15:22] <Geth> ¦ cro-http/link-gen-no-fork: 43902e077e | WIP

[15:22] <Geth> ¦ cro-http/link-gen-no-fork: 40a14fb9fc | Code cleanup (debug leftover, missing comments)

[15:22] <Geth> ¦ cro-http/link-gen-no-fork: dca10479b0 | Rework API a bit and do some minor cleanup

[15:22] <Geth> ¦ cro-http/link-gen-no-fork: review: https://github.com/croservices/cro-http/compare/13e7019d7c3f...dca10479b06c

[15:47] <Geth> ¦ cro-http/policy: 1bab432868 | Altai-man++ | lib/Cro/HTTP/Client.pm6

[15:47] <Geth> ¦ cro-http/policy: Properly initialize client's implicit timeout values

[15:47] <Geth> ¦ cro-http/policy: review: https://github.com/croservices/cro-http/commit/1bab432868

[15:56] <Geth> ¦ cro-http/link-gen-no-fork: 9f47a63cfd | (Jonathan Worthington)++ | 4 files

[15:56] <Geth> ¦ cro-http/link-gen-no-fork: Tweak link generator API and module name

[15:56] <Geth> ¦ cro-http/link-gen-no-fork: review: https://github.com/croservices/cro-http/commit/9f47a63cfd

[16:34] *** patrickb joined
[17:31] <Geth> ¦ cro-http/link-gen-no-fork: af011e2cb3 | Altai-man++ | 5 files

[17:31] <Geth> ¦ cro-http/link-gen-no-fork: A couple of tweaks for link generator

[17:31] <Geth> ¦ cro-http/link-gen-no-fork: review: https://github.com/croservices/cro-http/commit/af011e2cb3

[17:38] <Geth> ¦ cro-http/policy: 5e80153d34 | (Jonathan Worthington)++ | 3 files

[17:38] <Geth> ¦ cro-http/policy: Get response body timeout working correctly

[17:38] <Geth> ¦ cro-http/policy: 

[17:38] <Geth> ¦ cro-http/policy: The previous approach didn't correctly factor the time elapsed for the

[17:38] <Geth> ¦ cro-http/policy: connection and headers phase into the total, and also would not handle

[17:38] <Geth> ¦ cro-http/policy: persistent connections correctly. Do better by providing a mechanism for

[17:38] <Geth> ¦ cro-http/policy: a HTTP response to be cancelled - that is to say, terminated even before

[17:38] <Geth> ¦ cro-http/policy: the body has been fully received. Use this to enforce the timeout (and

[17:38] <Geth> ¦ cro-http/policy: gain a bonus feature along with it). Only implemented for HTTP/1.1 thus

[17:38] <Geth> ¦ cro-http/policy: far.

[17:38] <Geth> ¦ cro-http/policy: review: https://github.com/croservices/cro-http/commit/5e80153d34

[17:39] <Geth> ¦ cro-http/policy: d3db5dfbc2 | (Jonathan Worthington)++ | t/http-timeout.t

[17:39] <Geth> ¦ cro-http/policy: Test for total time working for body also

[17:39] <Geth> ¦ cro-http/policy: review: https://github.com/croservices/cro-http/commit/d3db5dfbc2

[17:46] <Geth> ¦ cro-http/link-gen-no-fork: c30fcbec31 | Altai-man++ | lib/Cro/HTTP/Router.pm6

[17:46] <Geth> ¦ cro-http/link-gen-no-fork: Better joiner for a warning

[17:46] <Geth> ¦ cro-http/link-gen-no-fork: review: https://github.com/croservices/cro-http/commit/c30fcbec31

[18:06] *** Altai-man left
[23:19] *** patrickb left
