-
Notifications
You must be signed in to change notification settings - Fork 159
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
Apply CSP rules for onlyoffice and richdocuments #5420
Conversation
Results for oCISSharingBasic https://drone.owncloud.com/owncloud/web/16962/47/1
|
Results for oCISSharingBasic https://drone.owncloud.com/owncloud/web/17003/47/1
|
Results for oC10Files1 https://drone.owncloud.com/owncloud/web/17003/10/1 💥 The acceptance tests pipeline failed. The build has been cancelled. |
Results for oCISSharingBasic https://drone.owncloud.com/owncloud/web/17009/47/1
|
Results for oC10Basic https://drone.owncloud.com/owncloud/web/17197/6/1 |
edd7b50
to
47bd0e9
Compare
Kudos, SonarCloud Quality Gate passed! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🐘
Results for oC10SharingInternalGroupsSharingIndicator https://drone.owncloud.com/owncloud/web/17203/23/1
|
Results for oCISSharingBasic https://drone.owncloud.com/owncloud/web/17203/49/1
|
Description
The iframe integrations of the documentserver coming from onlyoffice / richdocuments need certain CSP rules. With this PR we're extracting the respective documentserver domains from the two apps. We'll work on a generic solution when it becomes relevant.
Motivation and Context
Enable onlyoffice to build an integration of their documentserver into a web extension.
Types of changes
Checklist: