0perationPrivacy / VoIP

Web Base Telnyx/Twilio VoIP Interface enabling SMS, MMS and Voice calls using the providers API Keys and self-hosting the application.
https://VoIP.OperationPrivacy.com
GNU General Public License v3.0
246 stars 421 forks source link

Cannot start upgraded repo #131

Closed slfhstr closed 2 years ago

slfhstr commented 2 years ago

TypeError: Cannot read property 'trim' of undefined at Object. (/root/VoIP/app.js:101:23) at Module._compile (internal/modules/cjs/loader.js:778:30) at Object.Module._extensions..js (internal/modules/cjs/loader.js:789:10) at Module.load (internal/modules/cjs/loader.js:653:32) at tryModuleLoad (internal/modules/cjs/loader.js:593:12) at Function.Module._load (internal/modules/cjs/loader.js:585:3) at Function.Module.runMain (internal/modules/cjs/loader.js:831:12) at startup (internal/bootstrap/node.js:283:19) at bootstrapNodeJSCore (internal/bootstrap/node.js:623:3)



**To Reproduce**
As above
NB : direct deployment of repo in Ubuntu, not using Heroku

**Expected behavior**
App to start 
slfhstr commented 2 years ago

Oooops my error The format of .env has changed with additional declaration of HTTPS Copying across my previous .env missed this Add the HTTPS = false line and it is working

I don't understand why HTTPS is set to false Maybe it doesn't matter, as my nginx sets it to 443

0perationPrivacy commented 2 years ago

Oooops my error The format of .env has changed with additional declaration of HTTPS Copying across my previous .env missed this Add the HTTPS = false line and it is working

I don't understand why HTTPS is set to false Maybe it doesn't matter, as my nginx sets it to 443

As per the wiki: https://github.com/0perationPrivacy/voip/wiki/Cloud-Hosting#Environment-Variables

HTTPS default is false (for local testing without a certificate). Change it to true to force the page to redirect to https

Even if you don't change it and leave it at http, either your web server can force the redirect or your WAF (cloudflare, etc). But local access (for testing) will break because you can't issue a valid certificate on localhost.

All environment variables must be set, that's why I've provided a default file with some default values in it. They can all be overridden at runtime, they're just there for fallback in case you forget to set them.

Glad you got it to work šŸ‘šŸ»