plugin logLevel

[PabloSzx]

As it is right now, it is impossible to customize the logging level of the Next.js routes, even though the plugin type definitions have autoComplete for logLevel, and this PR reuses the plugin options logLevel config.

The test added fails if the changes in index.js are reverted.

[PabloSzx]

The tests are failling in CI but in local they are fine 🤔

[zekth]

The tests are failling in CI but in local they are fine 🤔

          diff: |
            --- expected
            +++ actual
            @@ -1,1 +1,1 @@
            -application/javascript; charset=UTF-8
            +text/html; charset=utf-8
          ...

May you missed committing something?

[PabloSzx]

The tests are failling in CI but in local they are fine 🤔

          diff: |
            --- expected
            +++ actual
            @@ -1,1 +1,1 @@
            -application/javascript; charset=UTF-8
            +text/html; charset=utf-8
          ...

May you missed committing something?

No, everything is fine, 56/56 passed, but the coverage is picking up the .next files, but I didn't change anything about that.

EDIT: I think some tests are failling (the /_next/* static tests) because of changes in dependencies, since the lock files are not commited, and I'm pretty sure that without my changes the tests would fail anyways.

[zekth]

Indeed. Your tests are ok as i see. Maybe another PR to fix this issue. Any idea @fastify/plugins ?

[mcollina]

A PR that fixes this would be welcomed, yes. Let's commit the lockfile for this one as it seems minor version of Next.js are breaking.

[PabloSzx]

Let's commit the lockfile for this one as it seems minor version of Next.js are breaking.

done 👍

[mcollina]

lgtm