Closed ttymothy closed 3 years ago
Hey @ttymothy
Thanks for this report. The Matrix notifiers is a recent contribution on Vigil that I didn't build myself, nor did I test it to be honest, as I didn't have any Matrix account at the time.
I've created one as to fix it, and decided to rework the notifier so that it is no more an optional feature, as it is now basically a zero-dependency notifier (except from the shared HTTP client).
Works like a charm w/ the fixes.
Issuing a Vigil release right now, a binary will be available within 30 minutes.
Commit ref: f0df4e05d23772e5e96e38b9cbadae31a31433ee
Update: a x64 binary is now available on https://github.com/valeriansaliou/vigil/releases/tag/v1.21.1
Thanks for the quick fix and release! The only drawback of not using the library is that messages are not e2e encrypted, but that's not a problem for me.
I've been testing Vigil (version 1.21.0), and I found it really good. However, when I enable the Matrix notifier, the process crashes almost instantly, with the error:
The configuration for the Matrix server may be wrong, so it may be a problem with the authentication, but I'd expect a nicer error message in that case. The only value that I have doubts about is the
device_id
, the rest are all correct. Can anyone help me troubleshoot this error? Searching online, the closest I could find was paritytech/polkadot#685, but I'm not sure if it's applicable here.Repro steps
The program must be compiled from source, as the prebuilt binaries were compiled without the
notifier-matrix
feature.wget https://github.com/valeriansaliou/vigil/archive/v1.21.0.tar.gz
cargo build --release --locked --all-features --target-dir=target
target/release/vigil -c issue.cfg
.Log and stack trace
Configuration file