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

adds an option to disable the setting of legacy headers #39

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

therephil
Copy link

As per MDN and OWASP it is recommended to not use deprecated security headers.
This PR enables passing that parameter to paragonie/csp-builder

Copy link

what-the-diff bot commented Dec 18, 2024

PR Summary

  • Introduction of 'Legacy' Option in Security Settings
    A new option named 'legacy' has been added in the security settings. This will help in managing the older and potentially deprecated security header generation.

  • Updation in the 'sendHeaders' method
    The 'sendHeaders' procedure has been enhanced to accept the 'legacy' option. This will influence the method of sending security header.

  • Default Setting of 'Legacy' Option
    The 'legacy' option is set to 'true' as the default setting. This means old security header generation is enabled automatically.

  • Readme Enhancement for 'Legacy headers'
    The readme file has been updated with a new section explaining 'Legacy headers'. This provides information on potential issues linked with deprecated headers and how they can be used.

  • Inclusion of 'Legacy' Option in Settings Table
    The new 'legacy' option has been included in the settings table detailed in the readme file. This allows users to understand its functionality better.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant