Skip to content

Commit 8e0687a

Browse files
bgiamarinoSachin KumarsachinkumarRP
authored
Remove Access Token Logging (#298)
* Removed access tokens from logs * Refactor access token code --------- Co-authored-by: Sachin Kumar <[email protected]> Co-authored-by: sachinkumarRP <[email protected]>
1 parent dcaa60a commit 8e0687a

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

app/code/Meta/BusinessExtension/Helper/GraphAPIAdapter.php

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -136,15 +136,19 @@ private function getUniqId()
136136
* @param array $request
137137
* @return \Psr\Http\Message\ResponseInterface
138138
* @throws GuzzleException
139-
* @todo implement custom logger class, remove access token from logs
139+
* @todo implement custom logger class
140140
*/
141141
private function callApi($method, $endpoint, $request)
142142
{
143143
try {
144+
$logRequest = $request;
144145
if ($this->debugMode) {
146+
if (isset($logRequest['access_token'])) {
147+
unset($logRequest['access_token']);
148+
}
145149
$this->logger->debug(json_encode([
146150
'endpoint' => "/{$method} {$endpoint}",
147-
'request' => $request,
151+
'request' => $logRequest,
148152
], JSON_PRETTY_PRINT));
149153
}
150154
// TODO: repalce with admin user local

0 commit comments

Comments
 (0)