Rewrite

[jamesli2021]

📚 Subject area/topic

Astro

📋 Page(s) affected (or suggested, for new content)

https://docs.astro.build/en/reference/configuration-reference/#experimentalrewriting

📋 Description of content that is out-of-date or incorrect

According to the docs, rewrite is out of experimental.

🖥️ Reproduction in StackBlitz (if reporting incorrect content or code samples)

NA

[sarah11918]

Hi! Can you let me know where you saw docs say this is out of experimental?

This feature is indeed still experimental, and the entry on the configuration reference page is correct! If there's somewhere else it we say it is NOT, then that's incorrect and should be fixed!

[jamesli2021]

@sarah11918 I'm a bit confused, my rewrite work and I did not set experimental.

[ematipico]

@sarah11918 I'm a bit confused, my rewrite work and I did not set experimental.

I probably forgot to add some warnings in the source code. I will take it from here. The feature is experimental, but I missed some checks. Thank you for flagging this.

[jamesli2021]

@sarah11918 I believe the blog shown an incorrect code as well?

https://astro.build/blog/astro-4100/#rewrite-for-all-http-methods

Is next('/next/v2') is the right API?

export const onRequest = defineMiddleware(({ request, url }, next) => {
  if(request.method === 'POST' && url.pathname === '/api') {
    return next('/api/v2');
  }
});

[ematipico]

It's one of the new APIs. The others are Astro.rewrite and context.rewrite