Security: Stores private host information in codebase. Suggest moving it to .env.local.

fireship-io / next13-pocketbase-demo

Demo for the new Next.js 13 features with Pocketbase.

276 stars 139 forks source link

Security: Stores private host information in codebase. Suggest moving it to .env.local. #27

Open StefanDeWolfe opened 9 months ago

StefanDeWolfe commented 9 months ago

Host address should be removed to .env.local for security. Just a nitpick, but it is good to practice correctly. Something like ${process.env.HOST} should be used in the string fields of the .tsx files and HOST=http://xxx.xxx.xxx.xxx:8090 in the .env.local file.