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

Various updates for Nanobox deploys #11988

Merged
merged 1 commit into from Oct 1, 2019

Conversation

@danhunsaker
Copy link
Collaborator

commented Sep 28, 2019

No description provided.

MAGICK_AREA_LIMIT=16MP
MAGICK_WIDTH_LIMIT=8KP
MAGICK_HEIGHT_LIMIT=8KP
BIND=0.0.0.0

This comment has been minimized.

Copy link
@Gargron

Gargron Sep 28, 2019

Member

Because Puma config is read before dotenv-rails is loaded, the BIND variable cannot be used from .env files 😕

This comment has been minimized.

Copy link
@Gargron

Gargron Sep 29, 2019

Member

I'm not sure if you need to bind to 0.0.0.0 anyway if you are running nginx in the same container / on the same machine?

@Gargron

This comment has been minimized.

Copy link
Member

commented Oct 1, 2019

Alright, it's probably not going to work the way you expect it to, but I suspect you may not even need to bind to 0.0.0.0, so I am just going to merge this as is.

@Gargron
Gargron approved these changes Oct 1, 2019
@Gargron Gargron merged commit 110bcf1 into tootsuite:master Oct 1, 2019
2 checks passed
2 checks passed
build-and-test Workflow: build-and-test
Details
codeclimate All good!
Details
hiyuki2578 added a commit to ProjectMyosotis/mastodon that referenced this pull request Oct 2, 2019
hiyuki2578 added a commit to ProjectMyosotis/mastodon that referenced this pull request Oct 2, 2019
@danhunsaker danhunsaker deleted the danhunsaker:fix/nanobox-provision-v3.0.0 branch Oct 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.