Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Disable 05-cspSsgPresets #349

Merged
merged 2 commits into from
Jan 17, 2024
Merged

Disable 05-cspSsgPresets #349

merged 2 commits into from
Jan 17, 2024

Conversation

vejja
Copy link
Collaborator

@vejja vejja commented Jan 17, 2024

Fixes #348
Fixes #335

Types of changes

  • Bug fix (a non-breaking change which fixes an issue)
  • New feature (a non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Description

This patch removes the cspSsgPresets functionality to temporarily solve issues #335 and #348, while we work on an alternative solution.

This patch is required after Nuxt 3.9.3 introduced protection against importing @nuxt/kit in Nitro plugins, which breaks 05-cspSsgPresets.ts in our module.

Checklist:

  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes (if not applicable, please state why)

Copy link

vercel bot commented Jan 17, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
nuxt-security ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 17, 2024 3:32pm

@Baroshem
Copy link
Owner

Some tests are not passing and was wondering if it is caused by the removing of the module or can we skip it?

@vejja
Copy link
Collaborator Author

vejja commented Jan 17, 2024

Some tests are not passing and was wondering if it is caused by the removing of the module or can we skip it?

Sorry I was too quick in hitting create
Tests now modified - passing alright now

@Baroshem
Copy link
Owner

Ok thanks for that @vejja !

I am off right now but will release a patch version in the evening with your change.

@Baroshem Baroshem merged commit bda8026 into main Jan 17, 2024
5 checks passed
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants