Google Groups Home
Help | Sign in
The lie our previewer tells
There are currently too many topics in this group that display first. To make this topic appear first, remove this option from another topic.
There was an error processing your request. Please try again.
flag
  6 messages - Collapse all
The group you are posting to is a Usenet group. Messages posted to this group will make your email address visible to anyone on the Internet.
Your reply message has not been sent.
Your post was successful
Ryan Mulligan  
View profile
 More options 26 Jun, 16:56
From: "Ryan Mulligan" <r...@ryantm.com>
Date: Thu, 26 Jun 2008 10:56:53 -0500
Local: Thurs 26 Jun 2008 16:56
Subject: The lie our previewer tells

I'm concerned that our previewer is lying to users. If you go to a URL
ending in / the previewer will add index.html onto the path for free. This
makes the assumption that people have webservers that route / to index.html
where they deploy the static files. This is not a good. Here are some
possible fixes:

1. Don't have the previewer append index.html
2. Warn users whenever previewer does this in the logs.
3. Warn users whenever the previewer starts
4. Warn users on the website.
5. provide a command line option for explicitly making the previewer do the
appending.

I think that I'm in favor of #5 it has the same correctness as #1 but is far
less annoying.


    Reply to author    Forward  
You must Sign in before you can post messages.
To post a message, you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
Stephen Bartholomew  
View profile
 More options 26 Jun, 18:17
From: Stephen Bartholomew <st...@curve21.com>
Date: Thu, 26 Jun 2008 10:17:54 -0700 (PDT)
Local: Thurs 26 Jun 2008 18:17
Subject: Re: The lie our previewer tells
Hey Ryan,

I don't think it's weird to *have* a default document like this.  I
think making user specify it manually each time would be annoying and
wouldn't match what you'd expect from a web server.

However, it's defiantly a good idea to allow users to configure it.
I'd be in favor of having some sensible defaults - namely index.html &
default.html (the defaults on apache & IIS) and allowing these to
added to configuration.  This would be in line with what I would
expect from web servers.

Thoughts?

Steve


    Reply to author    Forward  
You must Sign in before you can post messages.
To post a message, you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
Ryan Mulligan  
View profile
 More options 26 Jun, 19:56
From: "Ryan Mulligan" <r...@ryantm.com>
Date: Thu, 26 Jun 2008 13:56:17 -0500
Local: Thurs 26 Jun 2008 19:56
Subject: Re: The lie our previewer tells

okay, well if you are set on keeping it with defaults, I would say we need
to at least mention this in some deploying section of the website, possibly
we could provide a .htaccess file that explains how to do it, or at least
provide the snippet of mod_rewrite code for apache.

I noticed there is some new config code on tdreyno's branch which seems
nice. I guess we can put the DirectoryIndex configuration inside there.

On Thu, Jun 26, 2008 at 12:17 PM, Stephen Bartholomew <st...@curve21.com>
wrote:


    Reply to author    Forward  
You must Sign in before you can post messages.
To post a message, you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
Ryan Mulligan  
View profile
 More options 26 Jun, 22:55
From: "Ryan Mulligan" <r...@ryantm.com>
Date: Thu, 26 Jun 2008 16:55:01 -0500
Local: Thurs 26 Jun 2008 22:55
Subject: Re: The lie our previewer tells

Alright, well I just pushed a patch to github that implements this
DirectoryIndex feature we talked about.


    Reply to author    Forward  
You must Sign in before you can post messages.
To post a message, you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
Stephen Bartholomew  
View profile
 More options 27 Jun, 16:16
From: Stephen Bartholomew <st...@curve21.com>
Date: Fri, 27 Jun 2008 08:16:16 -0700 (PDT)
Local: Fri 27 Jun 2008 16:16
Subject: Re: The lie our previewer tells
Hey Ryan,

That's awesome - I'll check those all out shortly.

However, I'm still not entirely clear why you're not keen on directory
indexes.  What would you like to see coming up with you browse to the
previewer?

Steve


    Reply to author    Forward  
You must Sign in before you can post messages.
To post a message, you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
Ryan Mulligan  
View profile
 More options 28 Jun, 00:12
From: "Ryan Mulligan" <r...@ryantm.com>
Date: Fri, 27 Jun 2008 18:12:39 -0500
Local: Sat 28 Jun 2008 00:12
Subject: Re: The lie our previewer tells

The problem with the previewer doing DirectoryIndexes is that if you deploy
to somewhere that doesn't have DIrectoryIndices set up properly, your
website, and links will be broken. This happened to me once, and I didn't
notice right away because the previewer looked fine, but the deployed site
had some broken links.

Ryan Mulligan

On Fri, Jun 27, 2008 at 10:16 AM, Stephen Bartholomew <st...@curve21.com>
wrote:


    Reply to author    Forward  
You must Sign in before you can post messages.
To post a message, you must first join this group.
Please update your nickname on the subscription settings page before posting.
You do not have the permission required to post.
End of messages
« Back to Discussions « Newer topic     Older topic »

Create a group - Google Groups - Google Home - Terms of Service - Privacy Policy
©2008 Google