-
Notifications
You must be signed in to change notification settings - Fork 21
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
Add postgres in docker #42
Conversation
06b7c77
to
f9a75d3
Compare
Great, will merge when our Postgres is ready |
f9a75d3
to
ad3bfee
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Safety concerns
docker-compose.yml
Outdated
bot: | ||
hostname: bot | ||
image: flea | ||
environment: | ||
- REDIS_URL=redis | ||
- DATABASE_URL=postgres://postgres:123456@db:5432/postgres |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it safe to run a postgres
instance with such open credentials, given that we expose the 5432
port to external connections (line 16)?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You are right, its better to hide it, but I was waiting for postgres code to be merged. I will hide it here and also variables for db container as well
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mykolasolodukha fixed
@hexponent hey, if you could solve the conflicts and do the safety measure -- would be awesome! Will merge. Postgres is now activated within the bot |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
safety concern
ad3bfee
to
a3e3ad3
Compare
Thank you! @hexponent |
No description provided.