fix backward compatibility issue

[pheiduck]

Fixes #943
Closes #140
Closes #922
Thank you @utkarsh867

[pheiduck]

it's not fixing the issue at all:

 module.exports.WG_DEFAULT_ADDRESS = process.env.WG_DEFAULT_ADDRESS.repla   │
│   ce('x', '0') || '10.8.0.0';                                                │
│                                                                      ^       │
│   2024-03-27T13:39:58.376205588Z                                             │
│   TypeError: Cannot read properties of undefined (reading 'replace')         │
│       at Object.<anonymous> (/app/config.js:17:68)                           │
│       at Module._compile (node:internal/modules/cjs/loader:1356:14)          │
│       at Module._extensions..js (node:internal/modules/cjs/loader:1414:10)   │
│       at Module.load (node:internal/modules/cjs/loader:1197:32)              │
│       at Module._load (node:internal/modules/cjs/loader:1013:12)             │
│       at Module.require (node:internal/modules/cjs/loader:1225:19)           │
│       at require (node:internal/modules/helpers:177:18)                      │
│       at Object.<anonymous> (/app/lib/WireGuard.js:30:5)                     │
│       at Module._compile (node:internal/modules/cjs/loader:1356:14)          │
│       at Module._extensions..js (node:internal/modules/cjs/loader:1414:10)   │
│   2024-03-27T13:39:58.376485269Z                                             │
│   Node.js v18.19.1

[utkarsh867]

it's not fixing the issue at all:

 module.exports.WG_DEFAULT_ADDRESS = process.env.WG_DEFAULT_ADDRESS.repla   │
│   ce('x', '0') || '10.8.0.0';                                                │
│                                                                      ^       │
│   2024-03-27T13:39:58.376205588Z                                             │
│   TypeError: Cannot read properties of undefined (reading 'replace')         │
│       at Object.<anonymous> (/app/config.js:17:68)                           │
│       at Module._compile (node:internal/modules/cjs/loader:1356:14)          │
│       at Module._extensions..js (node:internal/modules/cjs/loader:1414:10)   │
│       at Module.load (node:internal/modules/cjs/loader:1197:32)              │
│       at Module._load (node:internal/modules/cjs/loader:1013:12)             │
│       at Module.require (node:internal/modules/cjs/loader:1225:19)           │
│       at require (node:internal/modules/helpers:177:18)                      │
│       at Object.<anonymous> (/app/lib/WireGuard.js:30:5)                     │
│       at Module._compile (node:internal/modules/cjs/loader:1356:14)          │
│       at Module._extensions..js (node:internal/modules/cjs/loader:1414:10)   │
│   2024-03-27T13:39:58.376485269Z                                             │
│   Node.js v18.19.1

that is strange. I did test it.

[utkarsh867]

Ah I see! In the case that WG_DEFAULT_ADDRESS is not defined, things break. oversight on my part. I can help solve this.

[pheiduck]

Have reverted #888 again. If we have a proper fix for issues which happend with that I can approve it.

[pheiduck]

I have - WG_DEFAULT_ADDRESS_IPV4=10.8.0.x
Also when using - WG_DEFAULT_ADDRESS=10.8.0.x it's not working

[utkarsh867]

WG_DEFAULT_ADDRESS_IPV4 is a valid configuration?

WG_DEFAULT_ADDRESS works on my end.

And I can connect to the VPN as well.

Result wg0 config:

[pheiduck]

WG_DEFAULT_ADDRESS_IPV4 is a valid configuration?

Yep, was from a diffrent fork I guess.
Works with WG_DEFAULT_ADDRESS also latest/12 works with that.