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

Security policy for the reverse proxy #52

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

Conversation

psavva
Copy link
Contributor

@psavva psavva commented Sep 14, 2023

Do not allow the browser to render the page inside an frame or iframe and avoid clickjacking
Allow iframes only from the same origin
Disable content-type sniffing
Cross-site scripting (XSS) filter
Enable HSTS(HTTP Strict Transport Security)
Do not leak server info such as the nginx version number in error pages or the server headers
Content Security Policy (CSP)

  • Note that this policy is not optimal. It since the indexer uses inline scripts, we have to allow "unsafe-inline" which can be imporoved upon by removing any inline scripts, either using a nonce a script hash or external file. It's however much safer than not doing anything at all.

@dangershony
Copy link
Member

It since the indexer uses inline scripts

You mean explorer?

This mean the proxy will be limited for other uses like wallet?

@psavva
Copy link
Contributor Author

psavva commented Sep 21, 2023

@dangershony The indexer UI has inline scrips which was being blocked.

@dangershony
Copy link
Member

I am fine with this @sondreb any comments? if not please merge

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.

2 participants