#31983: Add a warning if the file system cache location is within the static or
media roots
--------------------------------------+------------------------------------
     Reporter:  christa               |                    Owner:  nobody
         Type:  Cleanup/optimization  |                   Status:  new
    Component:  Core (Cache system)   |                  Version:  master
     Severity:  Normal                |               Resolution:
     Keywords:  cache check           |             Triage Stage:  Accepted
    Has patch:  1                     |      Needs documentation:  1
  Needs tests:  1                     |  Patch needs improvement:  1
Easy pickings:  1                     |                    UI/UX:  0
--------------------------------------+------------------------------------
Changes (by Johannes Hoppe):

 * cc: Johannes Hoppe (added)
 * needs_better_patch:  0 => 1
 * component:  Testing framework => Core (Cache system)
 * needs_tests:  0 => 1
 * version:  3.1 => master
 * easy:  0 => 1
 * needs_docs:  0 => 1
 * type:  Uncategorized => Cleanup/optimization
 * stage:  Unreviewed => Accepted


Old description:

> Hi, I have been reported an issue to security team one month ago. And the
> final conclusion is to check whether filesystem cache's  location is  a
> sub directory of special location.
>
> I wrote a check function and require a ticket to make a PR.

New description:

 Hi,

 I have been reported an issue regarding the file system cache location to
 security team one month ago. And the conclusion is to check whether file
 system cache's  location is  a subdirectory of special location.

 I wrote a check function and require a ticket to make a PR.

--

Comment:

 Hi there,

 You are making an excellent point. Since this seems to be your first code
 contribution, I recommend checking out the contribution guidelines to get
 you kick-started.

 Beyond that, I will do a review of your patch. Of the bat I can tell you
 that tests are missing. You will find examples for how to write a test by
 reviewing the tests for similar checks.

 Let me know if you need any pointers.

 Best,
 Joe

-- 
Ticket URL: <https://code.djangoproject.com/ticket/31983#comment:1>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/067.05db1a6a48f68adf5bbe1deb8cb9d102%40djangoproject.com.

Reply via email to