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

DOCS additional note to keeping uploaded files secure #933

Merged
merged 2 commits into from
Feb 2, 2020

Conversation

brynwhyman
Copy link

Touching up the documentation to advise how uploaded files can be kept secure.

Related (but does not address) #927

brynwhyman added 2 commits January 31, 2020 15:51
Touching up the documentation for userforms to advise how uploaded files can be kept secure.

Related (but does not address) silverstripe#927
@chillu
Copy link
Member

chillu commented Feb 2, 2020

I think we should implement #934 instead of pointing out that conceptual flaw in the docs :)

@brynwhyman
Copy link
Author

Arguably, the docs (userhelp and technical) are pointing this out already. I've just tried to wrap a little more context around this so readers understand the impact and a recommended action.

@chillu
Copy link
Member

chillu commented Feb 2, 2020

OK, I'll merge this - the follow up issue already contains a note that we need to adjust the docs once that bad default is fixed.

@chillu chillu merged commit f1ee7c8 into silverstripe:5 Feb 2, 2020
@chillu chillu deleted the securing-the-docs branch February 2, 2020 22:37
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