Re: [Django] #21080: collectstatic post-processing fails for references inside comments

[Django]

#21080: collectstatic post-processing fails for references inside comments
-+
 Reporter:  shreyas@…|Owner:  (none)
 Type:  Bug  |   Status:  new
Component:  contrib.staticfiles  |  Version:  dev
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  1|  Patch needs improvement:  1
Easy pickings:  0|UI/UX:  0
-+
Changes (by powderflask):

 * cc: powderflask (added)


-- 
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 view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/0107018b214be751-fec2b2a8-d3a8-4ecd-a88b-871830d01226-00%40eu-central-1.amazonses.com.

Re: [Django] #21080: collectstatic post-processing fails for references inside comments

[Django]

#21080: collectstatic post-processing fails for references inside comments
-+
 Reporter:  shreyas@…|Owner:  (none)
 Type:  Bug  |   Status:  new
Component:  contrib.staticfiles  |  Version:  dev
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  1|  Patch needs improvement:  1
Easy pickings:  0|UI/UX:  0
-+
Changes (by Petr Přikryl):

 * cc: Petr Přikryl (added)


-- 
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 view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/01070188ca08de18-dcdeb356-00fc-48c5-b439-c9d803da12f6-00%40eu-central-1.amazonses.com.

Re: [Django] #21080: collectstatic post-processing fails for references inside comments

[Django]

#21080: collectstatic post-processing fails for references inside comments
-+
 Reporter:  shreyas@…|Owner:  (none)
 Type:  Bug  |   Status:  new
Component:  contrib.staticfiles  |  Version:  dev
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  1|  Patch needs improvement:  1
Easy pickings:  0|UI/UX:  0
-+
Changes (by Manel Clos):

 * cc: Manel Clos (added)


-- 
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 view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/0107018858dbd25d-3089c2e5-745d-47c2-8ed0-29aaae0a7b81-00%40eu-central-1.amazonses.com.

Re: [Django] #21080: collectstatic post-processing fails for references inside comments

[Django]

#21080: collectstatic post-processing fails for references inside comments
-+
 Reporter:  shreyas@…|Owner:  (none)
 Type:  Bug  |   Status:  new
Component:  contrib.staticfiles  |  Version:  dev
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  1|  Patch needs improvement:  1
Easy pickings:  0|UI/UX:  0
-+
Changes (by Mariusz Felisiak):

 * owner:  Tomáš Zigo => (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 view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/01070188525cb28b-022b1def-c845-47f5-873a-58745ec9c2ce-00%40eu-central-1.amazonses.com.

Re: [Django] #21080: collectstatic post-processing fails for references inside comments

[Django]

#21080: collectstatic post-processing fails for references inside comments
-+-
 Reporter:  shreyas@…|Owner:  Tomáš
 |  Zigo
 Type:  Bug  |   Status:  assigned
Component:  contrib.staticfiles  |  Version:  dev
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  1|  Patch needs improvement:  1
Easy pickings:  0|UI/UX:  0
-+-

Comment (by Jake Bell):

 I'd also like to note that this bug also errors out when processing the
 CSS from Bootstrap 5, as it gets hung up on the SVG data URIs:

 {{{
 ValueError: The file '\"data:image/svg+xml,.
 }}}

 This seems like a pretty significant issue to me, given the popularity of
 Bootstrap. Simply parsing out the comments wouldn't be enough to fix the
 above issue. It seems like what is needed is either a proper CSS parser
 (like in the linked thread about JS files--perhaps
 [https://tinycss.readthedocs.io/]?) or a better regex that can exclude
 `url(data:)` items.

-- 
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 view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/010701874ec3245f-bc9a0421-73f6-4af5-bb4d-3f288ffe105f-00%40eu-central-1.amazonses.com.

Re: [Django] #21080: collectstatic post-processing fails for references inside comments

[Django]

#21080: collectstatic post-processing fails for references inside comments
-+-
 Reporter:  shreyas@…|Owner:  Tomáš
 |  Zigo
 Type:  Bug  |   Status:  assigned
Component:  contrib.staticfiles  |  Version:  dev
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  1|  Patch needs improvement:  1
Easy pickings:  0|UI/UX:  0
-+-

Comment (by SebastiaanZ):

 I noticed that the behavior got worse in Django 4.2.0b1 compared to
 pre-4.2 versions.

 Let's say you have a javascript file with these contents (minimized
 exampled):


 {{{
 function return_text(text_one, text_two) {
   return `${text_one} ${text_two}`;
 }


 class MyClass {
   render(){
 return return_text("import App from './App.vue'", 'import { Component
 } from "@org/components/loader";');
   }
 }
 }}}

 The "import" statements here are part of strings, not actually executable
 JavaScript code. In the original snippet, they're part of a component that
 shows information on how to use the component library and, as such,
 contains strings with imports as part of the documentation.

 In Django 3.2.x, 4.0.x, 4.1.x, running `collectstatic` works fine.

 However, in the new beta, you get an error:

 {{{
 (django-42-collectstatic-bug) D:\playground\django-42-collectstatic-
 bug>python manage.py collectstatic --clear --no-input -v 0
 Post-processing 'example.js' failed!

 Traceback (most recent call last):
   File "D:\playground\django-42-collectstatic-bug\manage.py", line 22, in
 
 main()
   File "D:\playground\django-42-collectstatic-bug\manage.py", line 18, in
 main
 execute_from_command_line(sys.argv)
   File "D:\virtualenvs\django-42-collectstatic-bug\lib\site-
 packages\django\core\management\__init__.py", line 442, in
 execute_from_command_line
 utility.execute()
   File "D:\virtualenvs\django-42-collectstatic-bug\lib\site-
 packages\django\core\management\__init__.py", line 436, in execute
 self.fetch_command(subcommand).run_from_argv(self.argv)
   File "D:\virtualenvs\django-42-collectstatic-bug\lib\site-
 packages\django\core\management\base.py", line 412, in run_from_argv
 self.execute(*args, **cmd_options)
   File "D:\virtualenvs\django-42-collectstatic-bug\lib\site-
 packages\django\core\management\base.py", line 458, in execute
 output = self.handle(*args, **options)
   File "D:\virtualenvs\django-42-collectstatic-bug\lib\site-
 packages\django\contrib\staticfiles\management\commands\collectstatic.py",
 line 209, in handle
 collected = self.collect()
   File "D:\virtualenvs\django-42-collectstatic-bug\lib\site-
 packages\django\contrib\staticfiles\management\commands\collectstatic.py",
 line 154, in collect
 raise processed
   File "D:\virtualenvs\django-42-collectstatic-bug\lib\site-
 packages\django\contrib\staticfiles\storage.py", line 364, in
 _post_process
 content = pattern.sub(converter, content)
   File "D:\virtualenvs\django-42-collectstatic-bug\lib\site-
 packages\django\contrib\staticfiles\storage.py", line 241, in converter
 hashed_url = self._url(
   File "D:\virtualenvs\django-42-collectstatic-bug\lib\site-
 packages\django\contrib\staticfiles\storage.py", line 174, in _url
 hashed_name = hashed_name_func(*args)
   File "D:\virtualenvs\django-42-collectstatic-bug\lib\site-
 packages\django\contrib\staticfiles\storage.py", line 414, in _stored_name
 cache_name = self.clean_name(self.hashed_name(name))
   File "D:\virtualenvs\django-42-collectstatic-bug\lib\site-
 packages\django\contrib\staticfiles\storage.py", line 135, in hashed_name
 raise ValueError(
 ValueError: The file 'App.vue'", 'import { Component } from
 "@org/components/loader' could not be found with
 .

 }}}

 Unfortunately, this a third-party dependency that we don't control. This
 means that we'd have to disable the manifest-backed static file storage
 completely in order to continue using this dependency (that we can't
 really get rid of), unless I've overlooked an option to disable import
 errors for specific files.

-- 
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 view this discussion on the web visit 

Re: [Django] #21080: collectstatic post-processing fails for references inside comments

[Django]

#21080: collectstatic post-processing fails for references inside comments
-+-
 Reporter:  shreyas@…|Owner:  Tomáš
 |  Zigo
 Type:  Bug  |   Status:  assigned
Component:  contrib.staticfiles  |  Version:  dev
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  1|  Patch needs improvement:  1
Easy pickings:  0|UI/UX:  0
-+-
Changes (by Claude Paroz):

 * Attachment "21080-pyparsing.patch" added.


-- 
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 view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/0107018664f8841a-4c26eb7f-8be0-441f-9a30-a06e12ba0908-00%40eu-central-1.amazonses.com.

Re: [Django] #21080: collectstatic post-processing fails for references inside comments

[Django]

#21080: collectstatic post-processing fails for references inside comments
-+-
 Reporter:  shreyas@…|Owner:  Tomáš
 |  Zigo
 Type:  Bug  |   Status:  assigned
Component:  contrib.staticfiles  |  Version:  dev
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  1|  Patch needs improvement:  1
Easy pickings:  0|UI/UX:  0
-+-

Comment (by Claude Paroz):

 I made a very quick and dirty experiment inside
 `HashedFilesMixin._post_process` using the `pyparsing` capability to
 remove comments from files (`pyparsing.cpp_style_comment.suppress()`).
 Running the Django staticfiles tests with or without that comments removal
 showed a performance penalty of ~330% (8.6 secs instead of 2.6 secs). A
 first optimization to only strip comments when an error occurs reduces the
 penalty to ~170% (strongly dependent on files at hand, of course).

 It is expected that using a parser instead of regexes will affect
 performance more or less badly. What performance penalty are we ready to
 pay for fixing this?

-- 
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 view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/0107018664f68a80-6f6f1fc1-17cb-4808-8376-7e5c690dfe85-00%40eu-central-1.amazonses.com.

Re: [Django] #21080: collectstatic post-processing fails for references inside comments

[Django]

#21080: collectstatic post-processing fails for references inside comments
-+-
 Reporter:  shreyas@…|Owner:  Tomáš
 |  Zigo
 Type:  Bug  |   Status:  assigned
Component:  contrib.staticfiles  |  Version:  dev
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  1|  Patch needs improvement:  1
Easy pickings:  0|UI/UX:  0
-+-

Comment (by Claude Paroz):

 If someone has time to make a little study on how we could use a real
 parser (e.g. [https://lark-parser.readthedocs.io/en/latest/ lark] or
 [https://pyparsing-docs.readthedocs.io/en/latest/ pyparsing]) to strip
 comments from CSS and JS, would be awesome! I'm sure a general parser
 could be used in other parts of Django as well.

-- 
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 view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/010701865c37644f-08c3284d-701c-49da-b7fb-8cd0fd642f50-00%40eu-central-1.amazonses.com.

Re: [Django] #21080: collectstatic post-processing fails for references inside comments

[Django]

#21080: collectstatic post-processing fails for references inside comments
-+-
 Reporter:  shreyas@…|Owner:  Tomáš
 |  Zigo
 Type:  Bug  |   Status:  assigned
Component:  contrib.staticfiles  |  Version:  dev
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  1|  Patch needs improvement:  1
Easy pickings:  0|UI/UX:  0
-+-

Comment (by Mariusz Felisiak):

 #34322 was a duplicate for ES module imports.

-- 
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 view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/010701865b7d9408-acc541e5-d1ed-49ab-a89a-c6bfca3f8576-00%40eu-central-1.amazonses.com.

Re: [Django] #21080: collectstatic post-processing fails for references inside comments

[Django]

#21080: collectstatic post-processing fails for references inside comments
-+-
 Reporter:  shreyas@…|Owner:  Tomáš
 |  Zigo
 Type:  Bug  |   Status:  assigned
Component:  contrib.staticfiles  |  Version:  dev
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  1|  Patch needs improvement:  1
Easy pickings:  0|UI/UX:  0
-+-

Comment (by Mariusz Felisiak ):

 In [changeset:"60be5909a29650a5e7595bbdc4012b0ad0c390af" 60be590]:
 {{{
 #!CommitTicketReference repository=""
 revision="60be5909a29650a5e7595bbdc4012b0ad0c390af"
 [4.1.x] Refs #21080, Refs #34322 -- Added warning to
 ManifestStaticFilesStorage docs about paths in comments.

 Backport of bae053d497ba8a8de7e4f725973924bfb1885fd2 from main.
 }}}

-- 
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 view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/010701865b798144-3b806b4b-1430-49b0-9bb7-54592b52d668-00%40eu-central-1.amazonses.com.

Re: [Django] #21080: collectstatic post-processing fails for references inside comments

[Django]

#21080: collectstatic post-processing fails for references inside comments
-+-
 Reporter:  shreyas@…|Owner:  Tomáš
 |  Zigo
 Type:  Bug  |   Status:  assigned
Component:  contrib.staticfiles  |  Version:  dev
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  1|  Patch needs improvement:  1
Easy pickings:  0|UI/UX:  0
-+-

Comment (by Mariusz Felisiak ):

 In [changeset:"e1c74bf45803cbde28149fbc80d91dc3b244e7fc" e1c74bf4]:
 {{{
 #!CommitTicketReference repository=""
 revision="e1c74bf45803cbde28149fbc80d91dc3b244e7fc"
 [4.2.x] Refs #21080, Refs #34322 -- Added warning to
 ManifestStaticFilesStorage docs about paths in comments.

 Backport of bae053d497ba8a8de7e4f725973924bfb1885fd2 from main.
 }}}

-- 
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 view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/010701865b777306-3a896351-aee7-4341-bf85-008b992558b3-00%40eu-central-1.amazonses.com.

Re: [Django] #21080: collectstatic post-processing fails for references inside comments

[Django]

#21080: collectstatic post-processing fails for references inside comments
-+-
 Reporter:  shreyas@…|Owner:  Tomáš
 |  Zigo
 Type:  Bug  |   Status:  assigned
Component:  contrib.staticfiles  |  Version:  dev
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  1|  Patch needs improvement:  1
Easy pickings:  0|UI/UX:  0
-+-

Comment (by GitHub ):

 In [changeset:"bae053d497ba8a8de7e4f725973924bfb1885fd2" bae053d4]:
 {{{
 #!CommitTicketReference repository=""
 revision="bae053d497ba8a8de7e4f725973924bfb1885fd2"
 Refs #21080, Refs #34322 -- Added warning to ManifestStaticFilesStorage
 docs about paths in comments.
 }}}

-- 
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 view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/010701865b738f1f-7a28de4c-651d-4f5f-b5f9-5f881614a027-00%40eu-central-1.amazonses.com.

Re: [Django] #21080: collectstatic post-processing fails for references inside comments

[Django]

#21080: collectstatic post-processing fails for references inside comments
-+-
 Reporter:  shreyas@…|Owner:  Tomáš
 |  Zigo
 Type:  Bug  |   Status:  assigned
Component:  contrib.staticfiles  |  Version:  dev
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  1|  Patch needs improvement:  1
Easy pickings:  0|UI/UX:  0
-+-

Comment (by Markus Bertheau):

 FWIW, the current approach of parsing CSS with regexes also doesn't work
 with SVG data inlined as data: URLs when they reference for example
 `..   `. This is changed to something like
 `fill='url("#someid#someid")'`.

 Two shortcomings here are:

 - double quotes are inserted that weren't there before, breaking the
 quoting of the data:-URL that the svg data is in
 - the doubling of the id fragment.

 Concerning the second problem: I didn't look too far into it, but that
 seems to happen because the font-face branch is triggered
 
https://github.com/django/django/blob/96f55ccf798c7592a1203f798a4dffaf173a9263/django/contrib/staticfiles/storage.py#L131-L142

 It's also at least curious that after line 131 final_url already has any
 fragment hash marks escaped (# -> %23), so urlsplit in line 137 doesn't
 catch it as a fragment.

 The relevant extract of our CSS looks like this:

 .sc_modal .content.branded-bg {
 background-image: url("data:image/svg+xml,%3csvg width='375' [...]
 %3cuse fill='url(%23c)' [...]");
 }

-- 
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 view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/075.0f093ec8c2bb2ec222abb1ae433e7b31%40djangoproject.com.

Re: [Django] #21080: collectstatic post-processing fails for references inside comments

[Django]

#21080: collectstatic post-processing fails for references inside comments
-+-
 Reporter:  shreyas@…|Owner:  Tomáš
 |  Zigo
 Type:  Bug  |   Status:  assigned
Component:  contrib.staticfiles  |  Version:  master
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  1|  Patch needs improvement:  1
Easy pickings:  0|UI/UX:  0
-+-
Changes (by Mariusz Felisiak):

 * needs_better_patch:  0 => 1
 * needs_tests:  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 view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/075.92559b888b9c5b1bbbc6e28f973ffbed%40djangoproject.com.

Re: [Django] #21080: collectstatic post-processing fails for references inside comments

[Django]

#21080: collectstatic post-processing fails for references inside comments
-+-
 Reporter:  shreyas@…|Owner:  Tomáš
 |  Zigo
 Type:  Bug  |   Status:  assigned
Component:  contrib.staticfiles  |  Version:  master
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-
Changes (by Jacob Walls):

 * owner:  Nathan Gaberel => Tomáš Zigo
 * needs_better_patch:  1 => 0


Comment:

 Resetting flags for more recent, unreviewed PR.

 [https://github.com/django/django/pull/12107 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 view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/075.75d8ea250ef8856cc647fc6450519abd%40djangoproject.com.

Re: [Django] #21080: collectstatic post-processing fails for references inside comments

[Django]

#21080: collectstatic post-processing fails for references inside comments
-+-
 Reporter:  shreyas@…|Owner:  Nathan
 |  Gaberel
 Type:  Bug  |   Status:  assigned
Component:  contrib.staticfiles  |  Version:  master
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  1
Easy pickings:  0|UI/UX:  0
-+-

Comment (by Wolph):

 Just in case anyone is looking for a _really_ simple hotfix, you can
 simply strip the quotes from the `name` in `hashed_name()`.

 Specifically, somewhere at the top of this method:
 
https://github.com/django/django/blob/efc3e32d6d7fb9bb41be73b80c8607b653c1fbd6/django/contrib/staticfiles/storage.py#L79-L111

 Or... put this monkeypatch somewhere in your code (settings file for
 example):
 {{{#!python
 from django.contrib.staticfiles import storage
 import functools

 original_hashed_name = storage.HashedFilesMixin.hashed_name

 @functools.wraps(original_hashed_name)
 def hashed_name(self, name, *args, **kwargs):
 return original_hashed_name(self, name.strip('"'), *args, **kwargs)

 storage.HashedFilesMixin.hashed_name = hashed_name
 }}}

-- 
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 view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/075.7cf11fc83f3f9eed5701b55a226264cc%40djangoproject.com.

Re: [Django] #21080: collectstatic post-processing fails for references inside comments

[Django]

#21080: collectstatic post-processing fails for references inside comments
-+-
 Reporter:  shreyas@…|Owner:  Nathan
 |  Gaberel
 Type:  Bug  |   Status:  assigned
Component:  contrib.staticfiles  |  Version:  master
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  1
Easy pickings:  0|UI/UX:  0
-+-
Changes (by felixxm):

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

Re: [Django] #21080: collectstatic post-processing fails for references inside comments

[Django]

#21080: collectstatic post-processing fails for references inside comments
-+-
 Reporter:  shreyas@…|Owner:  Nathan
 |  Gaberel
 Type:  Bug  |   Status:  assigned
Component:  contrib.staticfiles  |  Version:  master
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  1|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-
Changes (by Nathan Gaberel):

 * has_patch:  0 => 1


Comment:

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

 I put something together that detects block comments in css (using
 [https://github.com/django/django/pull/11241/files#diff-
 c7242dedd7c93b857a668acec1e310feR154 a regexp]) and ignores those urls
 that overlap with a comment.
 It has the benefit of not requiring additional dependencies to parse css.

 I think it's ready but I'm probably missing some test cases, feedback
 welcome!

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

Re: [Django] #21080: collectstatic post-processing fails for references inside comments

[Django]

#21080: collectstatic post-processing fails for references inside comments
-+-
 Reporter:  shreyas@…|Owner:  Nathan
 |  Gaberel
 Type:  Bug  |   Status:  assigned
Component:  contrib.staticfiles  |  Version:  master
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+-
Changes (by Nathan Gaberel):

 * status:  new => assigned
 * owner:  nobody => Nathan Gaberel


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

Re: [Django] #21080: collectstatic post-processing fails for references inside comments

[Django]

#21080: collectstatic post-processing fails for references inside comments
-+
 Reporter:  shreyas@…|Owner:  nobody
 Type:  Bug  |   Status:  new
Component:  contrib.staticfiles  |  Version:  master
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+

Comment (by powderflask):

 Replying to [comment:10 powderflask]:
 If you need to resolve this issue in the interim, override
 ManifestStaticFilesStorage (or CachedStaticFilesStorage) and re-define the
 class patterns property:

 {{{
 from django.contrib.staticfiles import storage

 class
 PatchedManifestStaticFilesStorage(storage.ManifestStaticFilesStorage):
 """
 Override the replacement patterns to match URL-encoded quotations.
 """
 patterns = (
 ("*.css", (
 r"""(url\((?:['"]|%22|%27){0,1}\s*(.*?)(?:['"]|%22|%27){0,1}\))""",
 (r"""(@import\s*["']\s*(.*?)["'])""", """@import
 url("%s")"""),
 )),
 )
 }}}

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

Re: [Django] #21080: collectstatic post-processing fails for references inside comments

[Django]

#21080: collectstatic post-processing fails for references inside comments
-+
 Reporter:  shreyas@…|Owner:  nobody
 Type:  Bug  |   Status:  new
Component:  contrib.staticfiles  |  Version:  master
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+

Comment (by Claude Paroz):

 As long as we are using regular expressions, excluding comment contents
 seems out of question. So your patch might be worthfile in the short term.
 A proper resolution of this ticket would be to use a JS parser to get
 links. This will be much more invasive 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/075.e8f4569afb09096d15e4c79487a664e4%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #21080: collectstatic post-processing fails for references inside comments

[Django]

#21080: collectstatic post-processing fails for references inside comments
-+
 Reporter:  shreyas@…|Owner:  nobody
 Type:  Bug  |   Status:  new
Component:  contrib.staticfiles  |  Version:  master
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+

Comment (by powderflask):

 Re: comment:6  is a slightly different issue and MUCH easier to patch.

 This issue is caused by generators (like http://jqueryui.com/themeroller)
 that urlencode stuff in a comment block.
 E.g.
 {{{
 /*! jQuery UI - v1.12.1 - 2016-09-14
   * http://jqueryui.com
   * Includes: core.css, accordion.css, ...
   * To view and modify this theme, visit
 http://jqueryui.com/themeroller/?...=url(%22images%2Fui-
 icons_55_256x240.png%22)...
   * Copyright jQuery Foundation and other contributors; Licensed MIT */
 }}}

 The regular expression in storage.HashedFilesMixin matches `url(...)` in
 the comment, but it does NOT unquote the urlencoded quotation mark: `%22`
 The last step of `converter()` unquotes the transformed url:  `return
 template % unquote(transformed_url)`,  converting the urlencoded quotation
 marks back, and thus messing up the filename, as can be seen in the posted
 stack trace:
 {{{
 The file 'jquery-ui-dist/"images/ui-icons_55_256x240.png"' could
 not be found...
 }}}
 (Notice the excess quotation marks in the path.)

 == Patch:
 storage.py line 61
 `-  r"""(url\(['"]{0,1}\s*(.*?)["']{0,1}\))""", `
 `+
 r"""(url\((?:['"]|%22|%27){0,1}\s*(.*?)(?:['"]|%22|%27){0,1}\))""", `

 This matches url encoded single- or double-quotes and fixed several such
 issues with 3rd party packages that included generated CSS files like
 this.

 IF a general fix for ignoring comments in CSS files is implemented, that
 would resolve this issue too.
 But IF NOT, this is a simple fix that prevents `collectstatic` from
 crashing on common 3rd party packages.

 Happy to submit a patch for this if this seems worthwhile.

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

Re: [Django] #21080: collectstatic post-processing fails for references inside comments

[Django]

#21080: collectstatic post-processing fails for references inside comments
-+
 Reporter:  shreyas@…|Owner:  nobody
 Type:  Bug  |   Status:  new
Component:  contrib.staticfiles  |  Version:  master
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+
Changes (by Joshua Smith):

 * version:  2.0 => master


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

Re: [Django] #21080: collectstatic post-processing fails for references inside comments

[Django]

#21080: collectstatic post-processing fails for references inside comments
-+
 Reporter:  shreyas@…|Owner:  nobody
 Type:  Bug  |   Status:  new
Component:  contrib.staticfiles  |  Version:  2.0
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+
Changes (by Joshua Smith):

 * cc: Joshua Smith (added)
 * version:  1.11 => 2.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/075.d9a92fb79e31e194df24687aea76297c%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Re: [Django] #21080: collectstatic post-processing fails for references inside comments

[Django]

#21080: collectstatic post-processing fails for references inside comments
-+
 Reporter:  shreyas@…|Owner:  nobody
 Type:  Bug  |   Status:  new
Component:  contrib.staticfiles  |  Version:  1.11
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+
Changes (by Udo Schochtert):

 * version:  1.5 => 1.11


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

Re: [Django] #21080: collectstatic post-processing fails for references inside comments

[Django]

#21080: collectstatic post-processing fails for references inside comments
-+
 Reporter:  shreyas@…|Owner:  nobody
 Type:  Bug  |   Status:  new
Component:  contrib.staticfiles  |  Version:  1.5
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+

Comment (by Udo Schochtert):

 hi, apparently this issue is still open in django 1.11.

 my deploy to heroku failed with the following error log:
 {{{#!shell
 ...
 $ python manage.py collectstatic --noinput
Post-processing 'jquery-ui-dist/jquery-ui.theme.css' failed!
Traceback (most recent call last):
  File "manage.py", line 10, in 
execute_from_command_line(sys.argv)
  File "/app/.heroku/python/lib/python3.6/site-
 packages/django/core/management/__init__.py", line 364, in
 execute_from_command_line
utility.execute()
  File "/app/.heroku/python/lib/python3.6/site-
 packages/django/core/management/__init__.py", line 356, in execute
self.fetch_command(subcommand).run_from_argv(self.argv)
  File "/app/.heroku/python/lib/python3.6/site-
 packages/django/core/management/base.py", line 283, in run_from_argv
self.execute(*args, **cmd_options)
  File "/app/.heroku/python/lib/python3.6/site-
 packages/django/core/management/base.py", line 330, in execute
output = self.handle(*args, **options)
  File "/app/.heroku/python/lib/python3.6/site-
 packages/django/contrib/staticfiles/management/commands/collectstatic.py",
 line 199, in handle
collected = self.collect()
  File "/app/.heroku/python/lib/python3.6/site-
 packages/django/contrib/staticfiles/management/commands/collectstatic.py",
 line 145, in collect
raise processed
whitenoise.storage.MissingFileError: The file 'jquery-ui-
 dist/"images/ui-icons_55_256x240.png"' could not be found with
 .
The CSS file 'jquery-ui-dist/jquery-ui.theme.css' references a file
 which could not be found:
  jquery-ui-dist/"images/ui-icons_55_256x240.png"
Please check the URL references in this CSS file, particularly any
relative paths which might be pointing to the wrong location.
  ! Error while running '$ python manage.py collectstatic --noinput'.
See traceback above for details.
You may need to update application code to resolve this error.
Or, you can disable collectstatic for this application:
   $ heroku config:set DISABLE_COLLECTSTATIC=1
https://devcenter.heroku.com/articles/django-assets
  ! Push rejected, failed to compile Python app.
  ! Push failed
 }}}

 the "collectstatic" command fails here:
 ... The file 'jquery-ui-dist/"images/ui-icons_55_256x240.png"' could
 not be found with ...
 because it tries to parse js comments.

 The file path should be
   * jquery-ui-dist/images/ui-icons_55_256x240.png
 and not
   * jquery-ui-dist/"images/ui-icons_55_256x240.png"

 other people on stackoverflow run inmto this issue as well:
   * https://stackoverflow.com/questions/41772144/bad-filename-formed-by-
 collectstatic-from-jquery-ui-1-12-0-min-css
   * https://stackoverflow.com/questions/40010040/django-pipeline-throwing-
 valueerror-the-file-could-not-be-found

 would be great if this issue could be fixed. thanks ;)

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

Re: [Django] #21080: collectstatic post-processing fails for references inside comments

[Django]

#21080: collectstatic post-processing fails for references inside comments
-+
 Reporter:  shreyas@…|Owner:  nobody
 Type:  Bug  |   Status:  new
Component:  contrib.staticfiles  |  Version:  1.5
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+

Comment (by benspaulding):

 Just as an additional datapoint and another thing to consider if it comes
 to parsing the CSS, I recently started working on a project where the
 designers used some CSS expressions for an IE-specific stylesheet. The
 JScript used in one expression puts a particularly crazy bit like
 {{{.replace('url("','').replace('")','')}}}. As marfire said, that
 {{{url}}} bit in a string raises an exception. Fortunately I can overcome
 this for my project with a simple sublcass to make a very small change to
 the regular expression used. But that is one other thing to consider if it
 is decided that the {{{CachedStaticFilesStorage}}} needs to be more
 robust.

-- 
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/075.0843896a04a783c3092094c87c405a0c%40djangoproject.com.
For more options, visit https://groups.google.com/groups/opt_out.

Re: [Django] #21080: collectstatic post-processing fails for references inside comments

[Django]

#21080: collectstatic post-processing fails for references inside comments
-+
 Reporter:  shreyas@…|Owner:  nobody
 Type:  Bug  |   Status:  new
Component:  contrib.staticfiles  |  Version:  1.5
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+

Comment (by glarrain):

 This bit me today. "Dumb" workaround: replace that `url` in your comments
 with `u r l`.

-- 
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/075.1ab230a078493135fba6b3c7a6760bc9%40djangoproject.com.
For more options, visit https://groups.google.com/groups/opt_out.

Re: [Django] #21080: collectstatic post-processing fails for references inside comments

[Django]

#21080: collectstatic post-processing fails for references inside comments
-+
 Reporter:  shreyas@…|Owner:  nobody
 Type:  Bug  |   Status:  new
Component:  contrib.staticfiles  |  Version:  1.5
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+

Comment (by pabluk):

 I'm agree with the [[comment:1|comment]] of @marfire, the correct way to
 do this is with a CSS parser, because there are many cases to be treated
 (multi-line comments, nested comments, broken comments, etc.)

 Adding an option to skip failed imports was reported in ticket:19650 and
 it was marked as a duplicated of ticket:18958.

 If you really need ignore these errors, you can create a subclass of
 `CachedFilesMixin` on your own project and add a `try...except` statement
 to handle these kind of exceptions.

-- 
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/075.fd7306db3055028bdaff073ae89d9bea%40djangoproject.com.
For more options, visit https://groups.google.com/groups/opt_out.

Re: [Django] #21080: collectstatic post-processing fails for references inside comments

[Django]

#21080: collectstatic post-processing fails for references inside comments
-+
 Reporter:  shreyas@…|Owner:  nobody
 Type:  Bug  |   Status:  new
Component:  contrib.staticfiles  |  Version:  1.5
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+

Comment (by anonymous):

 Could you recommend a path forward? Can we at least have it skip failed
 imports with a warning rather than break?

 -Shreyas

-- 
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/075.5befe46854333fcf8694379414ceb1eb%40djangoproject.com.
For more options, visit https://groups.google.com/groups/opt_out.

Re: [Django] #21080: collectstatic post-processing fails for references inside comments

[Django]

#21080: collectstatic post-processing fails for references inside comments
-+
 Reporter:  shreyas@…|Owner:  nobody
 Type:  Bug  |   Status:  new
Component:  contrib.staticfiles  |  Version:  1.5
 Severity:  Normal   |   Resolution:
 Keywords:   | Triage Stage:  Accepted
Has patch:  0|  Needs documentation:  0
  Needs tests:  0|  Patch needs improvement:  0
Easy pickings:  0|UI/UX:  0
-+
Changes (by marfire):

 * needs_better_patch:   => 0
 * stage:  Unreviewed => Accepted
 * needs_tests:   => 0
 * needs_docs:   => 0


Comment:

 `CachedFilesMixin` is using simple
 
[https://github.com/django/django/blob/6a6428a36f221446b17eaf4876e92d3db1781962/django/contrib/staticfiles/storage.py#L49
 regular expressions] to match `url` and `@import` statements, and so is
 picking up the ones inside comments. It would also have a problem if the
 text was inside a string.

 If it's possible to rewrite the regex to cover all the corner cases, that
 would be the easy fix. But I doubt it. The robust way would be to parse
 the css files properly, but that would mean rewriting a lot of code...

-- 
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/075.80631c3e1b8199555a16525db52f8ff8%40djangoproject.com.
For more options, visit https://groups.google.com/groups/opt_out.