Re: [Django] #29293: valid UUID is not a valid UUID

[Django]

#29293: valid UUID is not a valid UUID
-+-
 Reporter:  Jorge Alfaro |Owner:  nobody
 Type:   |   Status:  closed
  Cleanup/optimization   |
Component:  Database layer   |  Version:  1.11
  (models, ORM)  |
 Severity:  Normal   |   Resolution:  invalid
 Keywords:  UUIDField,   | Triage Stage:
  postgresql, uuid   |  Unreviewed
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-

Comment (by Tim Graham):

 I don't think Django should make any effort to work around broken
 configurations. If `isinstance(value, uuid.UUID)` doesn't work, other code
 may also not work.

-- 
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/066.dc51475e467fe854be83c2a2d4ae9a08%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #29293: valid UUID is not a valid UUID

[Django]

#29293: valid UUID is not a valid UUID
-+-
 Reporter:  Jorge Alfaro |Owner:  nobody
 Type:   |   Status:  closed
  Cleanup/optimization   |
Component:  Database layer   |  Version:  1.11
  (models, ORM)  |
 Severity:  Normal   |   Resolution:  invalid
 Keywords:  UUIDField,   | Triage Stage:
  postgresql, uuid   |  Unreviewed
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-

Comment (by Jorge Alfaro):

 I completly understand it's not a django problem, but the links posted
 don't really provide a solution just an explanation to the problem, and I
 think django can "recover" from this issue easily with the solution I
 provided, do you see any problem with the extra validation? (the str())

 Replying to [comment:1 Tim Graham]:
 > I don't think Django is at fault. From
 [https://groups.google.com/d/topic/django-users/ZGb8ofw1Ux8/discussion
 django-users]:
 >
 > > I think the problem comes from mod_wsgi. I had the same problem when
 virtualhost listen on multiple ports.
 > > Please look at this GH issue
 https://github.com/GrahamDumpleton/mod_wsgi/issues/206
 > > and try this by
 https://github.com/GrahamDumpleton/mod_wsgi/issues/206#issuecomment-305019825
 by Dumpleton.
 > >
 > > The real issue : isinstance(value, uuid.UUID) is failling because of
 the use of sub interpreters.
 > > Details in the mod_wsgi documentation :
 http://modwsgi.readthedocs.io/en/develop/user-guides/processes-and-
 threading.html#python-sub-interpreters and
 http://modwsgi.readthedocs.io/en/develop/configuration-
 directives/WSGIApplicationGroup.html.

-- 
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/066.cf234d206e85bb1b7daac53ed672f737%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #29293: valid UUID is not a valid UUID

[Django]

#29293: valid UUID is not a valid UUID
-+-
 Reporter:  Jorge Alfaro |Owner:  nobody
 Type:   |   Status:  closed
  Cleanup/optimization   |
Component:  Database layer   |  Version:  1.11
  (models, ORM)  |
 Severity:  Normal   |   Resolution:  invalid
 Keywords:  UUIDField,   | Triage Stage:
  postgresql, uuid   |  Unreviewed
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-
Changes (by Tim Graham):

 * status:  new => closed
 * resolution:   => invalid


Comment:

 I don't think Django is at fault. From [https://groups.google.com/d/topic
 /django-users/ZGb8ofw1Ux8/discussion django-users]:

 > I think the problem comes from mod_wsgi. I had the same problem when
 virtualhost listen on multiple ports.
 > Please look at this GH issue
 https://github.com/GrahamDumpleton/mod_wsgi/issues/206
 > and try this by
 https://github.com/GrahamDumpleton/mod_wsgi/issues/206#issuecomment-305019825
 by Dumpleton.
 >
 > The real issue : isinstance(value, uuid.UUID) is failling because of the
 use of sub interpreters.
 > Details in the mod_wsgi documentation :
 http://modwsgi.readthedocs.io/en/develop/user-guides/processes-and-
 threading.html#python-sub-interpreters and
 http://modwsgi.readthedocs.io/en/develop/configuration-
 directives/WSGIApplicationGroup.html.

-- 
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/066.6d817f42c7867d892ec161e2c5d5831b%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.