Skip to content

Added missing tmp/captchas directory#64

Merged
barryo merged 1 commit intoopensolutions:masterfrom
rozwell:captcha
Mar 4, 2014
Merged

Added missing tmp/captchas directory#64
barryo merged 1 commit intoopensolutions:masterfrom
rozwell:captcha

Conversation

@rozwell
Copy link
Copy Markdown
Contributor

@rozwell rozwell commented Mar 4, 2014

Without it captcha images aren't generated.

@barryo
Copy link
Copy Markdown
Member

barryo commented Mar 4, 2014

@rozwell I suspect you have a permissions issue. See this line which, as you can see, creates the captcha directory if it does not exist.

You web server should be able to write to all files under var.

I'll merge this anyway as there's no harm having it pre-created.

barryo added a commit that referenced this pull request Mar 4, 2014
Added missing tmp/captchas directory
@barryo barryo merged commit 5614cf9 into opensolutions:master Mar 4, 2014
@rozwell
Copy link
Copy Markdown
Contributor Author

rozwell commented Mar 4, 2014

They're pretty restrictive since we're in a test phase, thanks ;)

@barryo
Copy link
Copy Markdown
Member

barryo commented Mar 4, 2014

You may find other stuff breaking if the application cannot write to var/ 😟

@rozwell
Copy link
Copy Markdown
Contributor Author

rozwell commented Mar 4, 2014

No worries, just fixed permissions to make sure it wont happen 😃

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