logto-io / js

🤓 Logto JS SDKs.
https://docs.logto.io/quick-starts/
MIT License
61 stars 40 forks source link

bug: @logto/nuxt package is blocking Nuxt prerender function. #770

Closed kgierke closed 2 months ago

kgierke commented 2 months ago

Describe the bug

At the moment the @logto/nuxt module is throwing an error if the configuration keys haven't been changed. Therefore using prerenderer or @nuxt/content is not possible if no values have been provided.

Expected behavior

Nuxt build should work without problems.

How to reproduce?

Install @logto/nuxt and @nuxt/content and try to build the project without providing the configuration values.

Context

I think the error should rather be a warning so it isn't preventing Nuxt from building the application.

In a Docker context where the variables are set during runtime and not at build time it shouldn't be necessary to provide the logto configuration keys.

Screenshots

wangsijie commented 2 months ago

Hi @kgierke, I've successfully reproduced the error you reported. Thank you for bringing this to our attention. I'll discuss with the team about changing it as a "warning" instead.