Skip to content

Commit e586c8b

Browse files
authored
Merge pull request #20 from tolidano/master
Minor changes to README.md for clarity
2 parents e2e480a + 5907cb2 commit e586c8b

File tree

1 file changed

+5
-3
lines changed

1 file changed

+5
-3
lines changed

README.md

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -147,7 +147,7 @@ package:
147147
148148
provider:
149149
name: aws
150-
runtime: nodejs6.10 # Because this runs on CloudFront (lambda@edge) it must be 6.10
150+
runtime: nodejs6.10 # Because this runs on CloudFront (lambda@edge) it must be 6.10 or greater
151151
region: ${self:custom.region}
152152
stage: ${self:custom.stage}
153153
# Note that Lambda@Edge does not actually support environment variables for lambda
@@ -216,8 +216,10 @@ module.exports = {
216216
var req = evt.Records[0].cf.request;
217217
218218
if (req.uri && req.uri.length && req.uri.substring(req.uri.length - 1) === '/') {
219-
console.log('changing "%s" to "%s"', req.uri, req.uri + 'index.html');
220-
req.uri = req.uri + 'index.html';
219+
var uri = req.uri + 'index.html';
220+
221+
console.log('changing "%s" to "%s"', req.uri, uri);
222+
req.uri = uri;
221223
}
222224
223225
cb(null, req);

0 commit comments

Comments
 (0)