fix: exclude stackHeaders from entry data assignment

marcinlad · marcinlad · commit 247e5af5c755 · 2025-02-14T12:42:13.000+01:00
Previously, if an entry contained stackHeaders, it would overwrite the existing stack headers. If tokens were updated in the meantime, this could result in an authorization error.
diff --git a/lib/stack/contentType/entry/index.js b/lib/stack/contentType/entry/index.js
@@ -29,7 +29,8 @@ export function Entry (http, data) {
     if (this.apiVersion && !this.stackHeaders.api_version) {
       this.stackHeaders.api_version = this.apiVersion;
     }
-    Object.assign(this, cloneDeep(data.entry))
+    const { stackHeaders, ...entryData } = data.entry;
+    Object.assign(this, cloneDeep(entryData));
     this.urlPath = `/content_types/${this.content_type_uid}/entries/${this.uid}`
 
     /**

Original file line number	Diff line number	Diff line change
`@@ -29,7 +29,8 @@ export function Entry (http, data) {`
`29`	`29`	`if (this.apiVersion && !this.stackHeaders.api_version) {`
`30`	`30`	`this.stackHeaders.api_version = this.apiVersion;`
`31`	`31`	`}`
`32`		`- Object.assign(this, cloneDeep(data.entry))`
	`32`	`+ const { stackHeaders, ...entryData } = data.entry;`
	`33`	`+ Object.assign(this, cloneDeep(entryData));`
`33`	`34`	this.urlPath = `/content_types/${this.content_type_uid}/entries/${this.uid}`
`34`	`35`
`35`	`36`	`/**`