From 020ab20b1a9efa4fa31131524e4137f79185dff8 Mon Sep 17 00:00:00 2001
From: Nick Shine <shine.nick@gmail.com>
Date: Fri, 10 Apr 2020 00:42:53 -0500
Subject: [PATCH] fix: remove incorrect parameter reference in simpleurl.js

---
 README.md    | 21 ++++++++++++++++++++-
 simpleurl.js |  2 +-
 2 files changed, 21 insertions(+), 2 deletions(-)

diff --git a/README.md b/README.md
index ef5d2e0..90c089e 100755
--- a/README.md
+++ b/README.md
@@ -10,7 +10,7 @@
 > * Stripped down to focus on Microsoft Azure __Authentication__ and __Authorization__ only.
 > * Webpack config added to bundle the handler and dependencies in to a single file.
 > * A __zip__ of the bundled lambda (sans `config.json`) [is released][releases] via a GitHub Action for use in downstream [IaC] projects like [terraform-aws-lambda-edge-azure-auth].
-> * Simple-Url handling for default `index.html` and trailing slash redirects.
+> * Simple URL (a.k.a [pretty URLs]) handling for default `index.html` and trailing slash redirects ([see below](#simple-urls-and-trailing-slash-redirects)).
 > * Downstream terraform module for deployment ([terraform-aws-lambda-edge-azure-auth]).
 
 ## Description
@@ -59,6 +59,23 @@ be automatically generated).
 
 [Manual Deployment](https://github.com/Widen/cloudfront-auth/wiki/Manual-Deployment) __*or*__ [AWS SAM Deployment](https://github.com/Widen/cloudfront-auth/wiki/AWS-SAM-Deployment)
 
+## Simple URLs and Trailing Slash Redirects
+
+This lambda function has some options that enable CloudFront to behave similar to to most static
+site web servers. When CloudFront is backed by a private __S3__ bucket using an OAI ([Origin
+Access Identity]), default `index.html` files are not resolved unless given explicitly in the url
+
+For example, a request for `https://example.com/about/` tries to retrieve an object `about/` in
+the bucket, not `about/index.html`) 
+
+With the `simple_urls_enabled` option enabled (defaults to `true`):
+
+* `https://example.com/about/` will return the `https://example.com/about/index.html` object
+
+With `trailing_slash_redirects_enabled`:
+
+* `https://example.com/about` will trigger a `301` redirect to `https://example.com/about/`
+
 ## Testing
 
 Detailed instructions on testing your function can be found [in the Wiki](https://github.com/Widen/cloudfront-auth/wiki/Debug-&-Test).
@@ -82,6 +99,8 @@ See [CONTRIBUTING.md](Contributing.md).
 [node]:https://nodejs.org/en/
 [npm]:https://www.npmjs.com/
 [oai]:http://docs.aws.amazon.com/AmazonCloudFront/latest/DeveloperGuide/private-content-restricting-access-to-s3.html#private-content-creating-oai-console
+[origin access identity]:https://docs.aws.amazon.com/AmazonCloudFront/latest/DeveloperGuide/private-content-restricting-access-to-s3.html
 [openssl]:https://www.openssl.org
+[pretty urls]:https://gohugo.io/content-management/urls/#pretty-urls
 [terraform-aws-lambda-edge-azure-auth]:https://registry.terraform.io/modules/nickshine/lambda-edge-azure-auth/aws/
 [releases]:https://github.com/nickshine/lambda-edge-azure-auth/releases
diff --git a/simpleurl.js b/simpleurl.js
index 054efc4..ec456fe 100644
--- a/simpleurl.js
+++ b/simpleurl.js
@@ -16,7 +16,7 @@ const redirect = (request, callback) => {
 
 module.exports.handleIndexes = (uri) => {
   if (uri.endsWith('/')) {
-    console.log(`'index.html' appended to request.uri: ${request.uri}`);
+    console.log(`'index.html' appended to request.uri: ${uri}`);
     return `${uri}index.html`;
   }