Skip to content

Commit df0f0fd

Browse files
owsiaklowsiakl
authored
Header sanitization & code fixes (#82)
1 parent ac916d4 commit df0f0fd

30 files changed

+289
-279
lines changed

‎resources/php-amazon-selling-partner-api/api.mustache‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -94,7 +94,7 @@ final {{#operations}}class {{packageName}}SDK
9494
$sanitizedRequest = $request;
9595
9696
foreach ($this->configuration->loggingSkipHeaders() as $sensitiveHeader) {
97-
$sanitizedRequest = $request->withoutHeader($sensitiveHeader);
97+
$sanitizedRequest = $sanitizedRequest->withoutHeader($sensitiveHeader);
9898
}
9999

100100
$this->logger->log(
@@ -120,7 +120,7 @@ final {{#operations}}class {{packageName}}SDK
120120
$sanitizedResponse = $response;
121121
122122
foreach ($this->configuration->loggingSkipHeaders() as $sensitiveHeader) {
123-
$sanitizedResponse = $response->withoutHeader($sensitiveHeader);
123+
$sanitizedResponse = $sanitizedResponse->withoutHeader($sensitiveHeader);
124124
}
125125

126126
$this->logger->log(

‎src/AmazonPHP/SellingPartner/Api/APlusSDK.php‎

Lines changed: 20 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,7 @@ public function createContentDocument(AccessToken $accessToken, string $region,
101101
$sanitizedRequest = $request;
102102

103103
foreach ($this->configuration->loggingSkipHeaders() as $sensitiveHeader) {
104-
$sanitizedRequest = $request->withoutHeader($sensitiveHeader);
104+
$sanitizedRequest = $sanitizedRequest->withoutHeader($sensitiveHeader);
105105
}
106106

107107
$this->logger->log(
@@ -126,7 +126,7 @@ public function createContentDocument(AccessToken $accessToken, string $region,
126126
$sanitizedResponse = $response;
127127

128128
foreach ($this->configuration->loggingSkipHeaders() as $sensitiveHeader) {
129-
$sanitizedResponse = $response->withoutHeader($sensitiveHeader);
129+
$sanitizedResponse = $sanitizedResponse->withoutHeader($sensitiveHeader);
130130
}
131131

132132
$this->logger->log(
@@ -313,7 +313,7 @@ public function getContentDocument(AccessToken $accessToken, string $region, str
313313
$sanitizedRequest = $request;
314314

315315
foreach ($this->configuration->loggingSkipHeaders() as $sensitiveHeader) {
316-
$sanitizedRequest = $request->withoutHeader($sensitiveHeader);
316+
$sanitizedRequest = $sanitizedRequest->withoutHeader($sensitiveHeader);
317317
}
318318

319319
$this->logger->log(
@@ -338,7 +338,7 @@ public function getContentDocument(AccessToken $accessToken, string $region, str
338338
$sanitizedResponse = $response;
339339

340340
foreach ($this->configuration->loggingSkipHeaders() as $sensitiveHeader) {
341-
$sanitizedResponse = $response->withoutHeader($sensitiveHeader);
341+
$sanitizedResponse = $sanitizedResponse->withoutHeader($sensitiveHeader);
342342
}
343343

344344
$this->logger->log(
@@ -552,7 +552,7 @@ public function listContentDocumentAsinRelations(AccessToken $accessToken, strin
552552
$sanitizedRequest = $request;
553553

554554
foreach ($this->configuration->loggingSkipHeaders() as $sensitiveHeader) {
555-
$sanitizedRequest = $request->withoutHeader($sensitiveHeader);
555+
$sanitizedRequest = $sanitizedRequest->withoutHeader($sensitiveHeader);
556556
}
557557

558558
$this->logger->log(
@@ -577,7 +577,7 @@ public function listContentDocumentAsinRelations(AccessToken $accessToken, strin
577577
$sanitizedResponse = $response;
578578

579579
foreach ($this->configuration->loggingSkipHeaders() as $sensitiveHeader) {
580-
$sanitizedResponse = $response->withoutHeader($sensitiveHeader);
580+
$sanitizedResponse = $sanitizedResponse->withoutHeader($sensitiveHeader);
581581
}
582582

583583
$this->logger->log(
@@ -803,7 +803,7 @@ public function postContentDocumentApprovalSubmission(AccessToken $accessToken,
803803
$sanitizedRequest = $request;
804804

805805
foreach ($this->configuration->loggingSkipHeaders() as $sensitiveHeader) {
806-
$sanitizedRequest = $request->withoutHeader($sensitiveHeader);
806+
$sanitizedRequest = $sanitizedRequest->withoutHeader($sensitiveHeader);
807807
}
808808

809809
$this->logger->log(
@@ -828,7 +828,7 @@ public function postContentDocumentApprovalSubmission(AccessToken $accessToken,
828828
$sanitizedResponse = $response;
829829

830830
foreach ($this->configuration->loggingSkipHeaders() as $sensitiveHeader) {
831-
$sanitizedResponse = $response->withoutHeader($sensitiveHeader);
831+
$sanitizedResponse = $sanitizedResponse->withoutHeader($sensitiveHeader);
832832
}
833833

834834
$this->logger->log(
@@ -1020,7 +1020,7 @@ public function postContentDocumentAsinRelations(AccessToken $accessToken, strin
10201020
$sanitizedRequest = $request;
10211021

10221022
foreach ($this->configuration->loggingSkipHeaders() as $sensitiveHeader) {
1023-
$sanitizedRequest = $request->withoutHeader($sensitiveHeader);
1023+
$sanitizedRequest = $sanitizedRequest->withoutHeader($sensitiveHeader);
10241024
}
10251025

10261026
$this->logger->log(
@@ -1045,7 +1045,7 @@ public function postContentDocumentAsinRelations(AccessToken $accessToken, strin
10451045
$sanitizedResponse = $response;
10461046

10471047
foreach ($this->configuration->loggingSkipHeaders() as $sensitiveHeader) {
1048-
$sanitizedResponse = $response->withoutHeader($sensitiveHeader);
1048+
$sanitizedResponse = $sanitizedResponse->withoutHeader($sensitiveHeader);
10491049
}
10501050

10511051
$this->logger->log(
@@ -1252,7 +1252,7 @@ public function postContentDocumentSuspendSubmission(AccessToken $accessToken, s
12521252
$sanitizedRequest = $request;
12531253

12541254
foreach ($this->configuration->loggingSkipHeaders() as $sensitiveHeader) {
1255-
$sanitizedRequest = $request->withoutHeader($sensitiveHeader);
1255+
$sanitizedRequest = $sanitizedRequest->withoutHeader($sensitiveHeader);
12561256
}
12571257

12581258
$this->logger->log(
@@ -1277,7 +1277,7 @@ public function postContentDocumentSuspendSubmission(AccessToken $accessToken, s
12771277
$sanitizedResponse = $response;
12781278

12791279
foreach ($this->configuration->loggingSkipHeaders() as $sensitiveHeader) {
1280-
$sanitizedResponse = $response->withoutHeader($sensitiveHeader);
1280+
$sanitizedResponse = $sanitizedResponse->withoutHeader($sensitiveHeader);
12811281
}
12821282

12831283
$this->logger->log(
@@ -1468,7 +1468,7 @@ public function searchContentDocuments(AccessToken $accessToken, string $region,
14681468
$sanitizedRequest = $request;
14691469

14701470
foreach ($this->configuration->loggingSkipHeaders() as $sensitiveHeader) {
1471-
$sanitizedRequest = $request->withoutHeader($sensitiveHeader);
1471+
$sanitizedRequest = $sanitizedRequest->withoutHeader($sensitiveHeader);
14721472
}
14731473

14741474
$this->logger->log(
@@ -1493,7 +1493,7 @@ public function searchContentDocuments(AccessToken $accessToken, string $region,
14931493
$sanitizedResponse = $response;
14941494

14951495
foreach ($this->configuration->loggingSkipHeaders() as $sensitiveHeader) {
1496-
$sanitizedResponse = $response->withoutHeader($sensitiveHeader);
1496+
$sanitizedResponse = $sanitizedResponse->withoutHeader($sensitiveHeader);
14971497
}
14981498

14991499
$this->logger->log(
@@ -1677,7 +1677,7 @@ public function searchContentPublishRecords(AccessToken $accessToken, string $re
16771677
$sanitizedRequest = $request;
16781678

16791679
foreach ($this->configuration->loggingSkipHeaders() as $sensitiveHeader) {
1680-
$sanitizedRequest = $request->withoutHeader($sensitiveHeader);
1680+
$sanitizedRequest = $sanitizedRequest->withoutHeader($sensitiveHeader);
16811681
}
16821682

16831683
$this->logger->log(
@@ -1702,7 +1702,7 @@ public function searchContentPublishRecords(AccessToken $accessToken, string $re
17021702
$sanitizedResponse = $response;
17031703

17041704
foreach ($this->configuration->loggingSkipHeaders() as $sensitiveHeader) {
1705-
$sanitizedResponse = $response->withoutHeader($sensitiveHeader);
1705+
$sanitizedResponse = $sanitizedResponse->withoutHeader($sensitiveHeader);
17061706
}
17071707

17081708
$this->logger->log(
@@ -1906,7 +1906,7 @@ public function updateContentDocument(AccessToken $accessToken, string $region,
19061906
$sanitizedRequest = $request;
19071907

19081908
foreach ($this->configuration->loggingSkipHeaders() as $sensitiveHeader) {
1909-
$sanitizedRequest = $request->withoutHeader($sensitiveHeader);
1909+
$sanitizedRequest = $sanitizedRequest->withoutHeader($sensitiveHeader);
19101910
}
19111911

19121912
$this->logger->log(
@@ -1931,7 +1931,7 @@ public function updateContentDocument(AccessToken $accessToken, string $region,
19311931
$sanitizedResponse = $response;
19321932

19331933
foreach ($this->configuration->loggingSkipHeaders() as $sensitiveHeader) {
1934-
$sanitizedResponse = $response->withoutHeader($sensitiveHeader);
1934+
$sanitizedResponse = $sanitizedResponse->withoutHeader($sensitiveHeader);
19351935
}
19361936

19371937
$this->logger->log(
@@ -2139,7 +2139,7 @@ public function validateContentDocumentAsinRelations(AccessToken $accessToken, s
21392139
$sanitizedRequest = $request;
21402140

21412141
foreach ($this->configuration->loggingSkipHeaders() as $sensitiveHeader) {
2142-
$sanitizedRequest = $request->withoutHeader($sensitiveHeader);
2142+
$sanitizedRequest = $sanitizedRequest->withoutHeader($sensitiveHeader);
21432143
}
21442144

21452145
$this->logger->log(
@@ -2164,7 +2164,7 @@ public function validateContentDocumentAsinRelations(AccessToken $accessToken, s
21642164
$sanitizedResponse = $response;
21652165

21662166
foreach ($this->configuration->loggingSkipHeaders() as $sensitiveHeader) {
2167-
$sanitizedResponse = $response->withoutHeader($sensitiveHeader);
2167+
$sanitizedResponse = $sanitizedResponse->withoutHeader($sensitiveHeader);
21682168
}
21692169

21702170
$this->logger->log(

‎src/AmazonPHP/SellingPartner/Api/AuthorizationSDK.php‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,7 @@ public function getAuthorizationCode(AccessToken $accessToken, string $region, s
6868
$sanitizedRequest = $request;
6969

7070
foreach ($this->configuration->loggingSkipHeaders() as $sensitiveHeader) {
71-
$sanitizedRequest = $request->withoutHeader($sensitiveHeader);
71+
$sanitizedRequest = $sanitizedRequest->withoutHeader($sensitiveHeader);
7272
}
7373

7474
$this->logger->log(
@@ -93,7 +93,7 @@ public function getAuthorizationCode(AccessToken $accessToken, string $region, s
9393
$sanitizedResponse = $response;
9494

9595
foreach ($this->configuration->loggingSkipHeaders() as $sensitiveHeader) {
96-
$sanitizedResponse = $response->withoutHeader($sensitiveHeader);
96+
$sanitizedResponse = $sanitizedResponse->withoutHeader($sensitiveHeader);
9797
}
9898

9999
$this->logger->log(

‎src/AmazonPHP/SellingPartner/Api/CatalogItemSDK.php‎

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@ public function getCatalogItem(AccessToken $accessToken, string $region, string
7373
$sanitizedRequest = $request;
7474

7575
foreach ($this->configuration->loggingSkipHeaders() as $sensitiveHeader) {
76-
$sanitizedRequest = $request->withoutHeader($sensitiveHeader);
76+
$sanitizedRequest = $sanitizedRequest->withoutHeader($sensitiveHeader);
7777
}
7878

7979
$this->logger->log(
@@ -98,7 +98,7 @@ public function getCatalogItem(AccessToken $accessToken, string $region, string
9898
$sanitizedResponse = $response;
9999

100100
foreach ($this->configuration->loggingSkipHeaders() as $sensitiveHeader) {
101-
$sanitizedResponse = $response->withoutHeader($sensitiveHeader);
101+
$sanitizedResponse = $sanitizedResponse->withoutHeader($sensitiveHeader);
102102
}
103103

104104
$this->logger->log(
@@ -281,7 +281,7 @@ public function listCatalogCategories(AccessToken $accessToken, string $region,
281281
$sanitizedRequest = $request;
282282

283283
foreach ($this->configuration->loggingSkipHeaders() as $sensitiveHeader) {
284-
$sanitizedRequest = $request->withoutHeader($sensitiveHeader);
284+
$sanitizedRequest = $sanitizedRequest->withoutHeader($sensitiveHeader);
285285
}
286286

287287
$this->logger->log(
@@ -306,7 +306,7 @@ public function listCatalogCategories(AccessToken $accessToken, string $region,
306306
$sanitizedResponse = $response;
307307

308308
foreach ($this->configuration->loggingSkipHeaders() as $sensitiveHeader) {
309-
$sanitizedResponse = $response->withoutHeader($sensitiveHeader);
309+
$sanitizedResponse = $sanitizedResponse->withoutHeader($sensitiveHeader);
310310
}
311311

312312
$this->logger->log(
@@ -496,7 +496,7 @@ public function listCatalogItems(AccessToken $accessToken, string $region, strin
496496
$sanitizedRequest = $request;
497497

498498
foreach ($this->configuration->loggingSkipHeaders() as $sensitiveHeader) {
499-
$sanitizedRequest = $request->withoutHeader($sensitiveHeader);
499+
$sanitizedRequest = $sanitizedRequest->withoutHeader($sensitiveHeader);
500500
}
501501

502502
$this->logger->log(
@@ -521,7 +521,7 @@ public function listCatalogItems(AccessToken $accessToken, string $region, strin
521521
$sanitizedResponse = $response;
522522

523523
foreach ($this->configuration->loggingSkipHeaders() as $sensitiveHeader) {
524-
$sanitizedResponse = $response->withoutHeader($sensitiveHeader);
524+
$sanitizedResponse = $sanitizedResponse->withoutHeader($sensitiveHeader);
525525
}
526526

527527
$this->logger->log(

‎src/AmazonPHP/SellingPartner/Api/FBAInboundSDK.php‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -66,7 +66,7 @@ public function getItemEligibilityPreview(AccessToken $accessToken, string $regi
6666
$sanitizedRequest = $request;
6767

6868
foreach ($this->configuration->loggingSkipHeaders() as $sensitiveHeader) {
69-
$sanitizedRequest = $request->withoutHeader($sensitiveHeader);
69+
$sanitizedRequest = $sanitizedRequest->withoutHeader($sensitiveHeader);
7070
}
7171

7272
$this->logger->log(
@@ -91,7 +91,7 @@ public function getItemEligibilityPreview(AccessToken $accessToken, string $regi
9191
$sanitizedResponse = $response;
9292

9393
foreach ($this->configuration->loggingSkipHeaders() as $sensitiveHeader) {
94-
$sanitizedResponse = $response->withoutHeader($sensitiveHeader);
94+
$sanitizedResponse = $sanitizedResponse->withoutHeader($sensitiveHeader);
9595
}
9696

9797
$this->logger->log(

‎src/AmazonPHP/SellingPartner/Api/FBAInventorySDK.php‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -70,7 +70,7 @@ public function getInventorySummaries(AccessToken $accessToken, string $region,
7070
$sanitizedRequest = $request;
7171

7272
foreach ($this->configuration->loggingSkipHeaders() as $sensitiveHeader) {
73-
$sanitizedRequest = $request->withoutHeader($sensitiveHeader);
73+
$sanitizedRequest = $sanitizedRequest->withoutHeader($sensitiveHeader);
7474
}
7575

7676
$this->logger->log(
@@ -95,7 +95,7 @@ public function getInventorySummaries(AccessToken $accessToken, string $region,
9595
$sanitizedResponse = $response;
9696

9797
foreach ($this->configuration->loggingSkipHeaders() as $sensitiveHeader) {
98-
$sanitizedResponse = $response->withoutHeader($sensitiveHeader);
98+
$sanitizedResponse = $sanitizedResponse->withoutHeader($sensitiveHeader);
9999
}
100100

101101
$this->logger->log(

0 commit comments

Comments
 (0)