Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

config.js を config.default.js からコピーしてくるようにする + 環境変数対応 #7

Closed
mpyw opened this issue Sep 7, 2018 · 7 comments
Assignees
Labels
help wanted Extra attention is needed

Comments

@mpyw
Copy link
Owner

mpyw commented Sep 7, 2018

  • コピー作成する npm script を定義
  • ZIPアーカイブデプロイするときには config.default.js は除外する
  • Git管理から config.js は除外する
@mpyw mpyw changed the title config.js を config.js.default からコピーしてくるようにする config.js を config.default.js からコピーしてくるようにする Sep 7, 2018
@mpyw mpyw added the help wanted Extra attention is needed label Sep 7, 2018
@potato4d
Copy link

環境変数から引き回してくるほうが良い気がしているけれど、どうだろう

@miyabisun
Copy link
Collaborator

出来る限り環境変数から持ってくる設計にすべきでしょうね。
https://dev.classmethod.jp/cloud/aws/aws-lambda-env-variables/

現状YAMLではなくconfig.jsなんで、宣言しなくてもデフォルト設定で動いてくれるようなものはprocess.env.HOGE || 'default-value'みたいな感じにしておくのが良いんじゃないかと思います。

@mpyw
Copy link
Owner Author

mpyw commented Sep 10, 2018

なるほど Lambda にも環境変数機能やっぱりあるんですね…
であればそっちのほうが良さそうですね

@mpyw mpyw changed the title config.js を config.default.js からコピーしてくるようにする 環境変数から設定を読み込む Sep 10, 2018
@mpyw mpyw changed the title 環境変数から設定を読み込む config.js を config.default.js からコピーしてくるようにする + 環境変数対応 Sep 10, 2018
@mpyw
Copy link
Owner Author

mpyw commented Sep 10, 2018

但し,項目自体が増減する,サービスエイリアスや絵文字のマッピングなどに関しては環境変数でやるとちょっとつらい気がするので,併用のほうがいいんじゃないかと思いました。

Slackのチャンネルとかエンドポイントはもちろん環境変数から取れる感じのほうがベター

@gaogao-9
Copy link
Collaborator

gaogao-9 commented Sep 10, 2018

localstackの利用するサービスを制限するやつ環境変数で設定できるけどカンマ区切りで特に苦もなく出来てしまったので、設計次第では環境変数で普通に行けそう

@gaogao-9
Copy link
Collaborator

gaogao-9 commented Sep 10, 2018

https://github.com/localstack/localstack#configurations

これの SERVICES の変数の指定が参考になります(バリューに指定するサービス名をaws-cliの値を利用している点なども含めて)

@mpyw
Copy link
Owner Author

mpyw commented Sep 11, 2018

なるほど!

@mpyw mpyw closed this as completed Jul 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

4 participants