Frank Elsinga

> Let's clean this up (rustfmt & etc) Sorry for that part. Forgot to do rustfmt after doing the (apparently missing doctests) testing run

I don't know why the test for `mssql_server` added in https://github.com/testcontainers/testcontainers-rs-modules-community/pull/72 is failing. Cannot debug this locally. @kymmt90 I am assuming you are using this in your testing infra. Is...

The testcase failour seems to be because https://github.com/paupino/rust-decimal/issues/669 => needs new `rust-decimal`-release We could solve this by adding ` --locked` to the `cargo-hack` command. With that flag, the PR tests...

Should `with_initsql` be - ```rust fn with_initsql(mut self, initsql: impl AsRef) -> Self ``` or - ```rust fn with_initsql(mut self, initsql: &str) -> Self ```

Has been reported Upstream, I don't see a fix on our side until StuWerk IT fixes this > Lieber Herr Elsinga, > > danke für den Hinweis. Wir gehen der...

> This breaks the ping feature If the impelentation of a new monitor breaks an existing monitor, this is a merge blocker. I don't see how it could though. Could...

> I initially wanted to use the builtin js websocket library, but some non compliant websocket servers don't return the Sec-WebSocket-Accept header, which causes websocket to error out. Executing curl...

Please also make sure that you add a short little testcase to ensure that this monitor keeps working [here](https://github.com/louislam/uptime-kuma/tree/master/test/backend-test). Regressions in monioting types which are not often used are hard...

> It seems like Macos, Ubuntu, and Windows are unable to establish a insecure ws connection. These platforms fail the insecure testcase, everything else passes. For some reason generic ARM64...

Yes, I should have added this feature to the 2.1 milestone. It is approved and will be merged once we are able to are able to cut the 2.0 release.