[BUG] Please configure the secret using the SECRET environment variable

[yaohaizhou]

Describe the bug I want to deploy nostream on a server. Nostream always notice that "Please configure the secret using the SECRET environment variable" and cannot start the service.

To Reproduce Steps to reproduce the behavior:

1. Run command './scripts/start'

System (please complete the following information):

• OS: Ubuntu 22.04.1 LTS
• Platform: docker
• Version: Docker version 23.0.1, build a5ee5b1 node v12.22.9 nostream latest

Logs Attaching to nostream, nostream-cache, nostream-db, nostream-migrate nostream-cache | 1:C 19 Feb 2023 00:56:32.841 # oO0OoO0OoO0Oo Redis is starting oO0OoO0OoO0Oo nostream-cache | 1:C 19 Feb 2023 00:56:32.841 # Redis version=7.0.5, bits=64, commit=00000000, modified=0, pid=1, just started nostream-cache | 1:C 19 Feb 2023 00:56:32.841 # Configuration loaded nostream-cache | 1:M 19 Feb 2023 00:56:32.842 # Server initialized nostream-cache | 1:M 19 Feb 2023 00:56:32.842 # WARNING Your system is configured to use the 'xen' clocksource which might lead to degraded performance. Check the result of the [slow-clocksource] system check: run 'redis-server --check-system' to check if the system's clocksource isn't degrading performance. nostream-db | The files belonging to this database system will be owned by user "postgres". nostream-db | This user must also own the server process. nostream-db | nostream-db | The database cluster will be initialized with locale "en_US.utf8". nostream-db | The default database encoding has accordingly been set to "UTF8". nostream-db | The default text search configuration will be set to "english". nostream-db | nostream-db | Data page checksums are disabled. nostream-db | nostream-db | fixing permissions on existing directory /var/lib/postgresql/data ... ok nostream-db | creating subdirectories ... ok nostream-db | selecting dynamic shared memory implementation ... posix nostream-db | selecting default max_connections ... 100 nostream-db | selecting default shared_buffers ... 128MB nostream-db | selecting default time zone ... Etc/UTC nostream-db | creating configuration files ... ok nostream-db | running bootstrap script ... ok nostream-db | performing post-bootstrap initialization ... ok nostream-db | syncing data to disk ... ok nostream-db | nostream-db | nostream-db | Success. You can now start the database server using: nostream-db | nostream-db | pg_ctl -D /var/lib/postgresql/data -l logfile start nostream-db | nostream-db | initdb: warning: enabling "trust" authentication for local connections nostream-db | initdb: hint: You can change this by editing pg_hba.conf or using the option -A, or --auth-local and --auth-host, the next time you run initdb. nostream-db | waiting for server to start....2023-02-19 00:56:33.943 UTC [48] LOG: starting PostgreSQL 15.2 (Debian 15.2-1.pgdg110+1) on x86_64-pc-linux-gnu, compiled by gcc (Debian 10.2.1-6) 10.2.1 20210110, 64-bit nostream-db | 2023-02-19 00:56:33.947 UTC [48] LOG: listening on Unix socket "/var/run/postgresql/.s.PGSQL.5432" nostream-db | 2023-02-19 00:56:33.961 UTC [51] LOG: database system was shut down at 2023-02-19 00:56:33 UTC nostream-db | 2023-02-19 00:56:33.972 UTC [48] LOG: database system is ready to accept connections nostream-db | done nostream-db | server started nostream-db | CREATE DATABASE nostream-db | nostream-db | nostream-db | /usr/local/bin/docker-entrypoint.sh: ignoring /docker-entrypoint-initdb.d/* nostream-db | nostream-db | 2023-02-19 00:56:34.158 UTC [48] LOG: received fast shutdown request nostream-db | waiting for server to shut down....2023-02-19 00:56:34.163 UTC [48] LOG: aborting any active transactions nostream-db | 2023-02-19 00:56:34.165 UTC [48] LOG: background worker "logical replication launcher" (PID 54) exited with exit code 1 nostream-db | 2023-02-19 00:56:34.166 UTC [49] LOG: shutting down nostream-db | 2023-02-19 00:56:34.170 UTC [49] LOG: checkpoint starting: shutdown immediate nostream-db | 2023-02-19 00:56:34.319 UTC [49] LOG: checkpoint complete: wrote 918 buffers (0.7%); 0 WAL file(s) added, 0 removed, 0 recycled; write=0.105 s, sync=0.030 s, total=0.153 s; sync files=250, longest=0.022 s, average=0.001 s; distance=4217 kB, estimate=4217 kB nostream-db | 2023-02-19 00:56:34.331 UTC [48] LOG: database system is shut down nostream-db | done nostream-db | server stopped nostream-db | nostream-db | PostgreSQL init process complete; ready for start up. nostream-db | nostream-db | 2023-02-19 00:56:34.418 UTC [1] LOG: starting PostgreSQL 15.2 (Debian 15.2-1.pgdg110+1) on x86_64-pc-linux-gnu, compiled by gcc (Debian 10.2.1-6) 10.2.1 20210110, 64-bit nostream-db | 2023-02-19 00:56:34.418 UTC [1] LOG: listening on IPv4 address "0.0.0.0", port 5432 nostream-db | 2023-02-19 00:56:34.418 UTC [1] LOG: listening on IPv6 address "::", port 5432 nostream-db | 2023-02-19 00:56:34.427 UTC [1] LOG: listening on Unix socket "/var/run/postgresql/.s.PGSQL.5432" nostream-db | 2023-02-19 00:56:34.438 UTC [64] LOG: database system was shut down at 2023-02-19 00:56:34 UTC nostream-db | 2023-02-19 00:56:34.446 UTC [1] LOG: database system is ready to accept connections nostream-migrate | nostream-migrate | added 36 packages in 2s nostream-migrate | nostream-migrate | 3 packages are looking for funding nostream-migrate | run npm fund for details nostream-migrate | npm notice nostream-migrate | npm notice New minor version of npm available! 9.3.1 -> 9.5.0 nostream-migrate | npm notice Changelog: https://github.com/npm/cli/releases/tag/v9.5.0 nostream-migrate | npm notice Run npm install -g npm@9.5.0 to update! nostream-migrate | npm notice nostream-migrate | Batch 1 run: 20 migrations nostream-migrate exited with code 0 nostream | nostream | ███▄ █ ▒█████ ██████ ▄▄▄█████▓ ██▀███ ▓█████ ▄▄▄ ███▄ ▄███▓ nostream | ██ ▀█ █ ▒██▒ ██▒▒██ ▒ ▓ ██▒ ▓▒▓██ ▒ ██▒▓█ ▀▒████▄ ▓██▒▀█▀ ██▒ nostream | ▓██ ▀█ ██▒▒██░ ██▒░ ▓██▄ ▒ ▓██░ ▒░▓██ ░▄█ ▒▒███ ▒██ ▀█▄ ▓██ ▓██░ nostream | ▓██▒ ▐▌██▒▒██ ██░ ▒ ██▒░ ▓██▓ ░ ▒██▀▀█▄ ▒▓█ ▄░██▄▄▄▄██ ▒██ ▒██ nostream | ▒██░ ▓██░░ ████▓▒░▒██████▒▒ ▒██▒ ░ ░██▓ ▒██▒░▒████▒▓█ ▓██▒▒██▒ ░██▒ nostream | ░ ▒░ ▒ ▒ ░ ▒░▒░▒░ ▒ ▒▓▒ ▒ ░ ▒ ░░ ░ ▒▓ ░▒▓░░░ ▒░ ░▒▒ ▓▒█░░ ▒░ ░ ░ nostream | ░ ░░ ░ ▒░ ░ ▒ ▒░ ░ ░▒ ░ ░ ░ ░▒ ░ ▒░ ░ ░ ░ ▒ ▒▒ ░░ ░ ░ nostream | ░ ░ ░ ░ ░ ░ ▒ ░ ░ ░ ░ ░░ ░ ░ ░ ▒ ░ ░ nostream | ░ ░ ░ ░ ░ ░ ░ ░ ░ ░ nostream | v1.22.4 nostream | NIPs implemented: 1,2,4,9,11,12,15,16,20,22,26,28,33,40 nostream | Pay-to-relay disabled nostream | Payments provider: zebedee nostream | Please configure the secret using the SECRET environment variable.

[jilv220]

Don't use this commit. It is bugged.

[jilv220]

Just keep using the previous release.

[yaohaizhou]

@jilv220 Yes. It works. Thank you!

[okjodom]

@jilv220 ,

Don't use this commit. It is bugged.

this bug is reproducible on latest. Please reactivate? @zyhhhy

Also, if you have more context on what bugs there are on latest master, could you share details here?

I know the failing check was added in this commit introduced by #194

Just keep using the previous release.

This is a non-tirvial suggestion especially after running migrations on previous release

[jilv220]

@jilv220 ,

Don't use this commit. It is bugged.

this bug is reproducible on latest. Please reactivate? @zyhhhy

Also, if you have more context on what bugs there are on latest master, could you share details here?

I know the failing check was added in this commit introduced by #194

Just keep using the previous release.

This is a non-tirvial suggestion especially after running migrations on previous release

You can connect the database and delete 'change_invoice_id' and 'scale_balance_addition' from the knex_migration table. This should let you revert the migration back to the previous version.

[jilv220]

I can start the local fine, but once I start the production, I no longer receive subscriptions from the client.