Merge pull request #16788 from nextcloud/enh/no_response_constructor
Do not enforce the parent constructor of response to be called
This commit is contained in:
commit
1521da14ff
1 changed files with 8 additions and 10 deletions
|
@ -93,13 +93,9 @@ class Response {
|
|||
private $throttleMetadata = [];
|
||||
|
||||
/**
|
||||
* Response constructor.
|
||||
*
|
||||
* @since 17.0.0
|
||||
*/
|
||||
public function __construct() {
|
||||
$this->setContentSecurityPolicy(new EmptyContentSecurityPolicy());
|
||||
$this->setFeaturePolicy(new EmptyFeaturePolicy());
|
||||
}
|
||||
|
||||
/**
|
||||
|
@ -241,12 +237,8 @@ class Response {
|
|||
$this->lastModified->format(\DateTime::RFC2822);
|
||||
}
|
||||
|
||||
// Build Content-Security-Policy and use default if none has been specified
|
||||
if(is_null($this->contentSecurityPolicy)) {
|
||||
$this->setContentSecurityPolicy(new ContentSecurityPolicy());
|
||||
}
|
||||
$this->headers['Content-Security-Policy'] = $this->contentSecurityPolicy->buildPolicy();
|
||||
$this->headers['Feature-Policy'] = $this->featurePolicy->buildPolicy();
|
||||
$this->headers['Content-Security-Policy'] = $this->getContentSecurityPolicy()->buildPolicy();
|
||||
$this->headers['Feature-Policy'] = $this->getFeaturePolicy()->buildPolicy();
|
||||
|
||||
if($this->ETag) {
|
||||
$mergeWith['ETag'] = '"' . $this->ETag . '"';
|
||||
|
@ -296,6 +288,9 @@ class Response {
|
|||
* @since 8.1.0
|
||||
*/
|
||||
public function getContentSecurityPolicy() {
|
||||
if ($this->contentSecurityPolicy === null) {
|
||||
$this->setContentSecurityPolicy(new EmptyContentSecurityPolicy());
|
||||
}
|
||||
return $this->contentSecurityPolicy;
|
||||
}
|
||||
|
||||
|
@ -304,6 +299,9 @@ class Response {
|
|||
* @since 17.0.0
|
||||
*/
|
||||
public function getFeaturePolicy(): EmptyFeaturePolicy {
|
||||
if ($this->featurePolicy === null) {
|
||||
$this->setFeaturePolicy(new EmptyFeaturePolicy());
|
||||
}
|
||||
return $this->featurePolicy;
|
||||
}
|
||||
|
||||
|
|
Loading…
Reference in a new issue