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)



Subject: regexes in 'dirs' broken
Download (untitled) / with headers
text/plain 589b
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.
There are several tests for regexes in t/05dirs.t, is this still broken?
Download (untitled) / with headers
text/plain 209b
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.


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.