Почему environment variables undefined?

Question

[prolina]

Есть файл .env, содержащий переменные вида: REACT_APP_EMAIL = email REACT_APP_PASS = password

При вызове этих переменных, они undefined.

Структура проекта:


Вызываю переменные в файле server.js (при этом использую dotenv)

const dotenv = require('dotenv');
dotenv.config();

const express = require('express');
const bodyParser = require('body-parser');
const nodemailer = require('nodemailer');

const app = express();


app.use(bodyParser.json());
app.use(bodyParser.urlencoded({extended: true}));

app.post('/contacts', (req, res) => {
  console.log(req.body);
  console.log(process.env)
    nodemailer.createTestAccount((err, account) => {
        const htmlEmail = `
        <h3>Contact details</h3>
        <ul>
        <li>Name: ${req.body.name}</li>
        <li>Email: ${req.body.email}</li>
        </ul>
        <h3>Message</h3>
        <p>${req.body.message}</p>
        `;
        let transporter = nodemailer.createTransport({
            service: 'gmail',
            host: 'smtp.gmail.com',
            port: 587,
            secure: true,
            requireTLS: true,
            auth: {
                user: process.env.REACT_APP_EMAIL,
                pass: process.env.REACT_APP_PASS
            }
        })



        transporter.verify(function(error, success) {
            if (error) {
              console.log(error);
            } else {
              console.log("Server is ready to take our messages");
            }
          });
        let mailOptions = {
            from: '',
            to: '[e-mail]',
            subject: 'New message',
            text: req.body.message,
            html: htmlEmail
        }
        transporter.sendMail(mailOptions, (err, info) => {
            if(err) {
                return console.log(err)
            }


            console.log('Message sent: %s', info.message)
            // console.log('Message URL: %s', nodemailer.getTestMessageUrl(info))
        })
    })
})

const PORT = process.env.PORT || 3001;

if (process.env.NODE_ENV === 'production') {
    // Serve any static files
    app.use(express.static(path.join(__dirname, 'client/build')));

    // Handle React routing, return all requests to React app
    app.get('*', function(req, res) {
      res.sendFile(path.join(__dirname, 'client/build', 'index.html'));
    });
  }

app.listen(PORT, () => {
    console.log(`Server listening on port ${PORT}`);

})

При этом, если посмотреть в консоли в файле index.js(который в папке src), то переменные отображаются с нужным значением

Answer 1

[Александр Шохров]

Если правильно разглядел, файлы лежат так:
./server.js
./client/.env
./client/src/index.js
, а dotenv по умолчанию ищет .env в текущей директории, если не переназначить, судя по описанию.

Предположу, что index.js запускался из ./client, а server.js, скажем, из папки, где сам находится.

UPD Спойлер: .env лежал не в CWD, помогло указание пути до него через dotenv.config({ path: … })

Comments

[prolina]

Я пробовала перенести файл .env в папку client, но это не помогло

[Александр Шохров]

prolina, а до этого где он лежал? Какая рабочая папка (process.cwd()), когда вызывается server.js? Изначально .env рядом с server.jsклали? Попробуйте абсолютный путь:

dotenv.config({ path: '/full/custom/path/to/your/env/vars' });

Либо, скажем, если .env обычно лежит рядом с server.js:
dotenv.config({path: __dirname + '/.env'});

Или как там принято,

dotenv.config({path: path.resolve(__dirname, '.env')});

[prolina]

Александр, я попробовала

dotenv.config({ path: '/full/custom/path/to/your/env/vars' });

и всё получилось.
Спасибо огромное! Кучу времени потратила на это)

[Александр Шохров]

prolina, на здоровье:) Всё же рекомендую вариант с __dirname. Потому что прописывание абсолютного пути — это прибивание гвоздями, и если потребуется переименовать какие-нибудь папки, или перенести приложение на другую машину, можно забыть поправить абсолютный путь, и словить баги. Если только это не какой-то чётко задокументированный путь типа /etc/yourapp/.env.

[Igor Meshalkin]

prolina, а куда это писать не важно?