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

plugin/clnrest: security improvements #6686

Merged

Conversation

ShahanaFarooqui
Copy link
Collaborator

@ShahanaFarooqui ShahanaFarooqui commented Sep 15, 2023

  • Generating three certs (CA, client and server) if they do not exist already
  • Added rune authentication for websocket server as well
  • Added two new clnrest config options for CORS and CSP headers

Changelog-Fixed: clnrest Certificate generation
Changelog-Fixed: Added rune validations for websocket notifications
Changelog-Added: New configurable Content-Security-Policy (CSP) header for clnrest
Changelog-Added: New configurable Cross-Origin-Resource-Sharing(CSP) header for clnrest

Generate client, server and CA certs for https and trusted CA update
@ShahanaFarooqui ShahanaFarooqui linked an issue Sep 15, 2023 that may be closed by this pull request
Copy link
Contributor

@rustyrussell rustyrussell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ack 014fbf4

@rustyrussell rustyrussell merged commit 1a18f61 into ElementsProject:master Sep 19, 2023
39 checks passed
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.

Plugin clnrest improvements
2 participants