search

jagaapple / next-secure-headers

Sets secure response headers for Next.js.
MIT License
317 stars 13 forks source link

Modify `source` value in sample codes #33

Closed jagaapple closed 3 years ago

jagaapple commented 3 years ago

💩 Bug Report

A summary of the bug

Currently the readme is written source: "/(.*)" , but it does not working in some cases -- for example, an application uses Next.js built-in I18n routings.

Current behavior

Configured headers is not applied in some cases.

Expected behavior

It should be applied even if an application uses Next.js built-in I18n routings.

Additional context

Use "/:path*" instead.

jagaapple commented 3 years ago

This bug can be reproduced in Next.js 10 which uses I18n routes.