Plack::App::Directory lists "Parent Directory" and "../" #277

Closed
pstadt opened this Issue Mar 28, 2012 · 1 comment

Comments

Projects
None yet
2 participants
Contributor

pstadt commented Mar 28, 2012

While playing with Plack I used Plack::App::Directory to list a directory.
On Linux it lists "Parent Directory" hardcoded from @files and "../" from parsing the directory items. Only "." is skipped while looping over the items.

And on top level (PATH_INFO eq /) Parent Directory shoulden't be listet at all.

Owner

miyagawa commented Aug 13, 2012

sounds fair. PR welcome on this.

@miyagawa miyagawa closed this in 2a84ce3 Aug 13, 2012

miyagawa added a commit that referenced this issue Aug 14, 2012

Checking in changes prior to tagging of version 1.0002.
Changelog diff is:

diff --git a/Changes b/Changes
index 799d98c..d1144bf 100644
--- a/Changes
+++ b/Changes
@@ -1,5 +1,18 @@
 Go to http://github.com/plack/Plack/issues for the roadmap and known issues.

+1.0002  Mon Aug 13 17:04:25 PDT 2012
+    [NEW FEATURES]
+        - Added --no-default-middleware option to plackup #290
+
+    [BUG FIXES]
+        - Use C locale for AccessLog strftime #313
+        - Escape Plack::Request URI path using RFC 3986 definition (ssmccoy)
+
+    [IMPROVEMENTS]
+        - Documentation improvements (ether, Tom Heady)
+        - Skip displaying ".." in Plack::App::Directory #277
+        - Document load_class() doesn't validate user input. #285
+
 1.0001  Thu Jul 26 16:24:13 PDT 2012
     [INCOMPATIBLE CHANGES]
         - Deleted lots of code, methods and warnings that have been deprecated since 0.99
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment