Justin Leatherwood

[NextjsSite construct] add basePath support back into NextjsSite construct

7

Partially addresses https://github.com/sst/sst/issues/2964 The [original, now-deprecated version of the `NextjsSite` construct](https://github.com/sst/sst/blob/d6917d8ee581f016072155b1898e7c50ecc141e2/packages/sst/src/constructs/deprecated/NextjsSite.ts) (as far as I can tell, it's deprecated because the non-deprecated construct [uses SsrSite as a base](https://github.com/sst/sst/blob/d6917d8ee581f016072155b1898e7c50ecc141e2/packages/sst/src/constructs/NextjsSite.ts#L174), whereas [the...

[SsrSite/NextjsSite]: Add cdk server configuration to ImageOptimizationFunction in SsrSite base construct

2

`enhancement` (?) See https://github.com/sst/sst/issues/3607, this provides a fix for that issue in the NextjsSite construct where VPC configuration is only applied to the origins of `type: 'function'` (see [this](https://github.com/sst/sst/blob/master/packages/sst/src/constructs/SsrSite.ts#L1098C1-L1100) and...

In `getObject()` the function starts with the following code: ``` // This will get everything in the path following the mountpath var s3Key = decodeURIComponent(req.originalUrl.substr(req.baseUrl.length + 1)); // If the...

Fix for "s3 key with url like `/?foo=bar` with query on `/` path, leads to error when reading .createReadStream()"

1

If I have a defaultKey such as index.html, and I request a / route with a query string, e.g., /?foo=bar. If i do this, that first if statement thats supposed...

Firefox Console Error: The Components object is deprecated. It will soon be removed.

1

### Steps to reproduce Open ReactJS.Net project using [Webpack recommended setup ](https://reactjs.net/bundling/webpack.html) in firefox, look at the console. ------- Console error in firefox related to use of non-standard global object,...

NextjsSite Construct does not forward VPC configuration to the Image Optimization Function lambda

2

Lower environments of my application talk to internal-only DNS/resources. For this reason, I have configured my `NextjsSite` SST construct to attach my application to my VPC like so: ```ts const...

### Link to the code that reproduces this issue https://github.com/jplwood/nextjs-rewrite-notfound-repro ### To Reproduce 1. start the application with `npm run dev` **Demonstrate the fallback rewrite:** 3. go to home route...