Skip to content

[APIWAY-204] AWS SES를 이용한 이메일 전송 기능 구현#5

Open
kyewonseo wants to merge 1 commit intoApiWay:masterfrom
kyewonseo:master
Open

[APIWAY-204] AWS SES를 이용한 이메일 전송 기능 구현#5
kyewonseo wants to merge 1 commit intoApiWay:masterfrom
kyewonseo:master

Conversation

@kyewonseo
Copy link
Copy Markdown
Contributor

No description provided.

"port": "1883",
"host": "apiway-pubsub"
}
} No newline at end of file
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

config.json 파일은 없애지만,

  1. config.json.template 파일을 하나 만든다.
  2. private 한 정보 말고 다른 정보들은 모두 살려 놓는다.
    e.g.) mqtt의 각종 정보 등등
  3. AWS를 위한 config 는 빈 값으로 넣어놓는다.
    e.g.)
    "aws": { "ses": { "key": "<your key>", "select": "<your secret>", "region": "<your region>" } }

이 Project는 Opensource임을 염두에 두셔야 합니다.
다른 개발자가 자신의 key를 넣고 해보고 싶은데,
어떤 포맷으로 key를 넣어야 할 지, guide는 해 주어야 합니다.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

넵~
3번에 대해 질문이 있습니다.
config.json 파일은 gitignore에 추가하면 파일 자체가 안올라갈텐데
그럼 config.json 파일을 gitignore에서 제외하고 값들만 위에서 말씀하신것처럼 빈값으로 넣고 커밋할까요?

아니면 config.json파일은 gitignore에 추가하지만,
config.json파일의 정보들은 template에서 주석으로 가이드를 해준다.가 의미하신게 맞을까요?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

config.json을 gitignore에 추가하자는 의미는,
git으로 관리하지 말자는 의미이며,
따라서 repo에 push하지 말아야 합니다.
대신, config.json.template 파일을 git으로 관리를 하고,
실제 프로젝트를 사용하는 사람은
clone후에 config.json.template을 config.json으로 카피해서 실제 key값을 넣고 사용하도록 README에 가이드를 하자는 의미였습니다.

혹시, 이해 안되시면, 이젠 voice로 소통을 해야겠네요..

*/

var config = require('../config')
var config = require('../secret/config')
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

secret 폴더를 궂이 만드실 필요는 없을것 같아요...(다른 public한 정보들도 다 같이 들어가니까요..)

그냥 ../config를 유지 하면 좋겠습니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants