Re: [Django] #27346: Stop setting the Content-Length header in ConditionalGetMiddleware

[Django]

#27346: Stop setting the Content-Length header in ConditionalGetMiddleware
-+-
 Reporter:  Kevin Christopher|Owner:  Adam
  Henry  |  Malinowski
 Type:   |   Status:  closed
  Cleanup/optimization   |
Component:  HTTP handling|  Version:  1.10
 Severity:  Normal   |   Resolution:  fixed
 Keywords:   | Triage Stage:  Ready for
 |  checkin
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  1|UI/UX:  0
-+-
Changes (by Tim Graham ):

 * status:  assigned => closed
 * resolution:   => fixed


Comment:

 In [changeset:"37809b891eae2e73c7e80fbf5d8b9a7e86e82b54" 37809b89]:
 {{{
 #!CommitTicketReference repository=""
 revision="37809b891eae2e73c7e80fbf5d8b9a7e86e82b54"
 Fixed #27346 -- Stopped setting the Content-Length header in
 ConditionalGetMiddleware.
 }}}

--
Ticket URL: 
Django 
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 django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/065.8055c1ef756cd7c123bd002115888382%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #27346: Stop setting the Content-Length header in ConditionalGetMiddleware

[Django]

#27346: Stop setting the Content-Length header in ConditionalGetMiddleware
-+-
 Reporter:  Kevin Christopher|Owner:  Adam
  Henry  |  Malinowski
 Type:   |   Status:  assigned
  Cleanup/optimization   |
Component:  HTTP handling|  Version:  1.10
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Ready for
 |  checkin
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  1|UI/UX:  0
-+-
Changes (by Tim Graham):

 * stage:  Accepted => Ready for checkin


--
Ticket URL: 
Django 
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 django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/065.8abcb2ae693d325c7762e38e810b81e9%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #27346: Stop setting the Content-Length header in ConditionalGetMiddleware

[Django]

#27346: Stop setting the Content-Length header in ConditionalGetMiddleware
-+-
 Reporter:  Kevin Christopher|Owner:  Adam
  Henry  |  Malinowski
 Type:   |   Status:  assigned
  Cleanup/optimization   |
Component:  HTTP handling|  Version:  1.10
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  1|UI/UX:  0
-+-
Changes (by Adam Malinowski):

 * needs_better_patch:  1 => 0


--
Ticket URL: 
Django 
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 django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/065.05085987633f4f046e64a44f77d7d7d8%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #27346: Stop setting the Content-Length header in ConditionalGetMiddleware

[Django]

#27346: Stop setting the Content-Length header in ConditionalGetMiddleware
-+-
 Reporter:  Kevin Christopher|Owner:  Adam
  Henry  |  Malinowski
 Type:   |   Status:  assigned
  Cleanup/optimization   |
Component:  HTTP handling|  Version:  1.10
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  1
Easy pickings:  1|UI/UX:  0
-+-
Changes (by Simon Charette):

 * needs_better_patch:  0 => 1


--
Ticket URL: 
Django 
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 django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/065.a00a48a855216055a3aaa8b7d5316836%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #27346: Stop setting the Content-Length header in ConditionalGetMiddleware

[Django]

#27346: Stop setting the Content-Length header in ConditionalGetMiddleware
-+-
 Reporter:  Kevin Christopher|Owner:  Adam
  Henry  |  Malinowski
 Type:   |   Status:  assigned
  Cleanup/optimization   |
Component:  HTTP handling|  Version:  1.10
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  1|UI/UX:  0
-+-
Changes (by Adam Malinowski):

 * has_patch:  0 => 1


Comment:

 [https://github.com/django/django/pull/7476: PR]

--
Ticket URL: 
Django 
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 django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/065.a45fa373e7014f5288c196569eef8d38%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #27346: Stop setting the Content-Length header in ConditionalGetMiddleware

[Django]

#27346: Stop setting the Content-Length header in ConditionalGetMiddleware
-+-
 Reporter:  Kevin Christopher|Owner:
  Henry  |  adammalinowski
 Type:   |   Status:  assigned
  Cleanup/optimization   |
Component:  HTTP handling|  Version:  1.10
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  1|UI/UX:  0
-+-
Changes (by adammalinowski):

 * status:  new => assigned
 * owner:  (none) => adammalinowski


--
Ticket URL: 
Django 
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 django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/065.3be4fc14c2d3fe945bc70a56415bf2c7%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #27346: Stop setting the Content-Length header in ConditionalGetMiddleware

[Django]

#27346: Stop setting the Content-Length header in ConditionalGetMiddleware
-+-
 Reporter:  Kevin Christopher|Owner:  (none)
  Henry  |
 Type:   |   Status:  new
  Cleanup/optimization   |
Component:  HTTP handling|  Version:  1.10
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  1|UI/UX:  0
-+-
Changes (by Martino Pizzol):

 * owner:  Martino Pizzol => (none)
 * status:  assigned => new


--
Ticket URL: 
Django 
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 django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/065.fdc1ef00a027404450c6eb1493d6d981%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #27346: Stop setting the Content-Length header in ConditionalGetMiddleware

[Django]

#27346: Stop setting the Content-Length header in ConditionalGetMiddleware
-+-
 Reporter:  Kevin Christopher|Owner:  Martino
  Henry  |  Pizzol
 Type:   |   Status:  assigned
  Cleanup/optimization   |
Component:  HTTP handling|  Version:  1.10
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  1|UI/UX:  0
-+-
Changes (by Martino Pizzol):

 * owner:  Aleksandr Sobolev => Martino Pizzol


--
Ticket URL: 
Django 
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 django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/065.928b5f555e94a03ea3703f48a8b7c67d%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #27346: Stop setting the Content-Length header in ConditionalGetMiddleware

[Django]

#27346: Stop setting the Content-Length header in ConditionalGetMiddleware
-+-
 Reporter:  Kevin Christopher|Owner:  Aleksandr
  Henry  |  Sobolev
 Type:   |   Status:  assigned
  Cleanup/optimization   |
Component:  HTTP handling|  Version:  1.10
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  1|UI/UX:  0
-+-
Changes (by Aleksandr Sobolev):

 * owner:  nobody => Aleksandr Sobolev
 * status:  new => assigned


--
Ticket URL: 
Django 
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 django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/065.41acdc92c1c9a36e0aaf68c126d02184%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #27346: Stop setting the Content-Length header in ConditionalGetMiddleware

[Django]

#27346: Stop setting the Content-Length header in ConditionalGetMiddleware
-+-
 Reporter:  Kevin Christopher|Owner:  nobody
  Henry  |
 Type:   |   Status:  new
  Cleanup/optimization   |
Component:  HTTP handling|  Version:  1.10
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  1|UI/UX:  0
-+-
Changes (by Tim Graham):

 * needs_better_patch:   => 0
 * stage:  Unreviewed => Accepted
 * type:  Uncategorized => Cleanup/optimization
 * needs_tests:   => 0
 * needs_docs:   => 0


Comment:

 It was [https://github.com/django/django/pull/6798#issuecomment-227557147
 discussed briefly] when adding `Content-Length` to `CommonMiddleware`. We
 didn't see much advantage to removing it at the time as
 `ConditionalGetMiddleware` might be used without `CommonMiddleware`. I
 don't have a strong feeling about it though.

--
Ticket URL: 
Django 
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 django-updates+unsubscr...@googlegroups.com.
To post to this group, send email to django-updates@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/065.3367882a77a8c1f32a019ae590aa25c3%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.