Skip to content
This repository has been archived by the owner on May 23, 2019. It is now read-only.

Add Content-Security-Policy meta tag #132

Merged
merged 1 commit into from
Jan 23, 2017
Merged

Add Content-Security-Policy meta tag #132

merged 1 commit into from
Jan 23, 2017

Conversation

ghys
Copy link
Member

@ghys ghys commented Jan 23, 2017

Helps mitigate XSS attacks inside expressions and through lazy
loading of external scripts by templates/custom widgets.

With this CSP policy, only local scripts can be loaded
(provided the browser has support for CSP).

WARNING: the user is still responsible for securing the
openHAB instance properly, especially the REST API endpoints
which modify data (POST /service/org.openhab.habpanel/config in
particular), including custom code, and never publish them
unsecured on the Internet.

HABPanel lets the user modify templates, and implicitly trusts
the code in these templates, therefore XSS attacks and arbitrary
code execution is always possible even with the expression sandbox!

See: https://docs.angularjs.org/guide/security

Signed-off-by: Yannick Schaus habpanel@schaus.net

Helps mitigate XSS attacks inside expressions and through lazy
loading of external scripts by templates/custom widgets.

With this CSP policy, only local scripts can be loaded
(provided the browser has support for CSP).

WARNING: the user is still responsible for securing the
openHAB instance properly, especially the REST API endpoints
which modify data (POST /service/org.openhab.habpanel/config in
particular), including custom code, and *never* publish them
unsecured on the Internet.

HABPanel lets the user modify templates, and implicitly trusts
the code in these templates, therefore XSS attacks and arbitrary
code execution is always possible even with the expression sandbox!

See: https://docs.angularjs.org/guide/security

Signed-off-by: Yannick Schaus <habpanel@schaus.net>
@ghys ghys merged commit 1f80200 into openhab:master Jan 23, 2017
@ghys ghys added this to the 2.1.0 milestone Jan 23, 2017
@ghys ghys deleted the csp branch April 25, 2017 18:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant