Repository: svix/svix-webhooks (language: rust)

Issue:
Unable to start svix-server without redis
I am trying to run svix server without redis, but not able to run server when I remove redis. In documentation it is given redis_dsn field can be left empty but even after changing redis_dsn to empty still server is not starting and giving this error.
```
backend_1    | thread 'main' panicked at 'Invalid Redis DSN', svix-server/src/wait_for.rs:9:55
backend_1    | note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
server_backend_1 exited with code 101
```


Discussion / hints:
Thanks for the report, and sorry for the delay, I missed your ticket!

You just need to remove/comment out the config var and it'll work.

I'm closing it, as it works as expected, but let me know if it doesn't and I'll reopen.
From what I can tell this doesn't work when using the docker image because the environment variables get merged with the default `config.default.toml` so `redis_dsn` always ends up being `redis://redis:6379`.

https://github.com/svix/svix-webhooks/blob/57c583d76ab9034663a0d89c71059d944eca87a2/server/svix-server/src/cfg.rs#L351-L356

https://github.com/svix/svix-webhooks/blob/57c583d76ab9034663a0d89c71059d944eca87a2/server/svix-server/src/cfg.rs#L77

https://github.com/svix/svix-webhooks/blob/57c583d76ab9034663a0d89c71059d944eca87a2/server/svix-server/config.default.toml#L55
As a workaround I made sure to set `SVIX_QUEUE_TYPE` to `memory` and set the `SVIX_REDIS_DSN` to the database URL. This way the `WAIT_FOR` waits for the database twice and then goes ahead and ignores redis.
@FSMaxB , I think your diagnosis is correct. At a minimum, we should probably leave the `redis_dsn` commented out in `config.default.toml`.
Oh, I completely misunderstood what was going on originally. James is 100% right, we should comment out the redis dsn in the default config.

Fix patch:
diff --git a/server/svix-server/config.default.toml b/server/svix-server/config.default.toml
index 20d734798..1b80f8cbf 100644
--- a/server/svix-server/config.default.toml
+++ b/server/svix-server/config.default.toml
@@ -49,10 +49,10 @@ db_dsn = "postgresql://postgres:postgres@pgbouncer/postgres"
 # Higher values can significantly increase performance if your database can handle it.
 db_pool_max_size = 20
 
-# The defualt DSN for redis. `queue_dsn` and `cache_dsn` with take precedence over this value.
+# The default DSN for redis. `queue_dsn` and `cache_dsn` with take precedence over this value.
 # (can be left empty if not using redis or if Redis is configured through the queue and/or cache
 # specific DSNs)
-redis_dsn = "redis://redis:6379"
+# redis_dsn = "redis://redis:6379"
 
 # The maximum number of connections for the Redis pool. Minimum value of 10
 # Higher values can significantly increase performance if your database can handle it.


Test patch: