Changeset 507 in code for trunk/config/config.go

Timestamp:

Apr 21, 2021, 4:15:04 PM (4 years ago)

Author:

contact

Message:

Make db and log config options more future-proof

Rename the "sql" directive to "db". Rename the "log" directive to
"log fs".

In the future, we'll maybe support more databases and more message
stores. Make it so it's easy to integrate these new festures to the
config file format.

File:

• trunk/config/config.go (modified) (1 diff)

trunk/config/config.go

@@ -87 +87 @@
                         }
                         srv.TLS = tls
-                case "sql":
+                case "db":
                         if err := d.ParseParams(&srv.SQLDriver, &srv.SQLSource); err != nil {
                                 return nil, err
                         }
                 case "log":
-                        if err := d.ParseParams(&srv.LogPath); err != nil {
+                        var driver string
+                        if err := d.ParseParams(&driver, &srv.LogPath); err != nil {
                                 return nil, err
+                        }
+                        if driver != "fs" {
+                                return nil, fmt.Errorf("directive %q: unknown driver %q", d.Name, driver)
                         }
                 case "http-origin":