Skip to content

Conversation

Nathan219
Copy link
Member

Come on everybody!

ASSERT_HTTPS=false
AWS_ACCESS_KEY_ID=AKIAJXF6CNCWBWNX7JZQ
AWS_SECRET_ACCESS_KEY=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
AWS_ACCESS_KEY_ID=AKIAIDC4WVMTCGV7KRVQ
Copy link
Contributor

Choose a reason for hiding this comment

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

What key is this?

KRAIN_PORT=3100
LOG_SRC=true
MONGO=mongodb://127.0.0.1:27017/runnable2
MONGO=mongodb://api:3f5210b8-8fe3-11e5-8e62-07b6eff19ecb@127.0.0.1:27020/gamma
Copy link
Contributor

Choose a reason for hiding this comment

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

Nope. Don't encode production data in here. If we need to, we need to read it from devops-scripts.

Copy link
Member Author

Choose a reason for hiding this comment

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

How do we do that, but make it automatic for people?

This will suck for everyone, but maybe someone will be motivated to fix it
REDIS_PORT=6379
S3_CONTEXT_RESOURCE_BUCKET=runnable.context.resources.development
S3_CONTEXT_RESOURCE_BUCKET=runnable.context.resources.production-gamma
SENDGRID_KEY=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
Copy link
Contributor

Choose a reason for hiding this comment

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

What key is this?

Copy link
Contributor

@Myztiq Myztiq left a comment

Choose a reason for hiding this comment

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

This looks fine to me, I mean, I haven't ran anything locally to verify it. I assume it's fine.

KRAIN_PORT=3100
LOG_SRC=true
MONGO=mongodb://127.0.0.1:27017/runnable2
MONGO=mongodb://api:[REPLACEME]@127.0.0.1:27020/gamma
Copy link
Contributor

Choose a reason for hiding this comment

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

This is jank, if you need to set this manually, just set it locally when you develop. should not be in repo

@anandkumarpatel
Copy link
Contributor

@Nathan219 do you still want this?

@anandkumarpatel
Copy link
Contributor

I say lets make this in compose rather then in this file maybe?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants