Skip Menu |

This queue is for tickets about the Catalyst-Plugin-Static-Simple CPAN distribution.

Report information
The Basics
Id: 85082
Status: open
Priority: 0/
Queue: Catalyst-Plugin-Static-Simple
People
Owner: Nobody in particular
Requestors: MAUKE [...] cpan.org
Cc:
AdminCc:
Bug Information
Severity: (no value)
Broken in:
  • 0.15
  • 0.16
  • 0.17
  • 0.18
  • 0.19
  • 0.20
  • 0.21
  • 0.22
  • 0.23
  • 0.24
  • 0.25
  • 0.26
  • 0.27
  • 0.28
  • 0.29
  • 0.30
Fixed in: (no value)

History Show all quoted text
  Mon May 06 08:09:42 2013 MAUKE [...] cpan.org - Ticket created
Subject: regexes in 'dirs' broken
According to the documentation the entries in 'dirs' can be either static directory names (strings) or regexes (qr// objects). This is wrong. Up to version 0.14 the module didn't actually support strings and treated everything as a regex. Version 0.15 "fixed" this by disabling regexes and treating everything as a fixed string instead. The test that's supposed to make sure qr/.../ works is broken: it sets dirs => [qr/^images/] and requests '/images/catalyst.png'. This succeeds because the filename contains a '.', so it's served by C:P:S:S no matter what directories are configured.
  Fri Jun 15 18:32:46 2018 abraxxa [...] cpan.org - Correspondence added
There are several tests for regexes in t/05dirs.t, is this still broken?
  Fri Jun 15 18:32:48 2018 The RT System itself - Status changed from 'new' to 'open'
  Mon Nov 12 14:08:05 2018 MYSOCIETY [...] cpan.org - Correspondence added
On Fri Jun 15 18:32:46 2018, ABRAXXA wrote: Show quoted text
> There are several tests for regexes in t/05dirs.t, is this still broken?
Yes, replace the 'always-static' line with qr/always-static/ and see how tests now fail.