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

Add a note about reporting security issues #697

Merged
merged 1 commit into from Jun 3, 2014
Merged

Conversation

@allait
Copy link
Contributor

@allait allait commented Apr 17, 2014

Since @kmike created a mailing list for security issues we should direct people there from the documentation.
Wording taken mostly from the Django docs: https://docs.djangoproject.com/en/dev/internals/contributing/bugs-and-features/#reporting-security-issues

@kmike
Copy link
Member

@kmike kmike commented Apr 17, 2014

+1. It may be a good idea to add this info to https://github.com/scrapy/scrapy.github.io as well.

@allait
Copy link
Contributor Author

@allait allait commented Apr 17, 2014

@kmike http://scrapy.org/community/ links to this documentation page as a place to start

@kmike
Copy link
Member

@kmike kmike commented Apr 17, 2014

Yes, but it also links to github repo and to scrapy-users mailing lists, and raising issue on github or sending message to a scrapy-users mailling list is natural enough - not everyone reads contributing guides.

dangra added a commit that referenced this pull request Jun 3, 2014
Add a note about reporting security issues
@dangra dangra merged commit 9830efb into scrapy:master Jun 3, 2014
1 check passed
1 check passed
continuous-integration/travis-ci The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants