Skip to content

Commit

Permalink
Disable verbose logging (fix #45) while #47 is WIP (#49)
Browse files Browse the repository at this point in the history
Comment out the verbose error logging segment. Plan is to expose this via option as part of Issue #47 and Pull #42
  • Loading branch information
lsthompson committed Jun 24, 2024
1 parent 76fabeb commit d8f744a
Showing 1 changed file with 7 additions and 8 deletions.
15 changes: 7 additions & 8 deletions pve2_api.class.php
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,6 @@
Proxmox VE APIv2 (PVE2) Client - PHP Class
Copyright (c) 2012-2014 Nathan Sullivan
Permission is hereby granted, free of charge, to any person obtaining a copy of
this software and associated documentation files (the "Software"), to deal in
the Software without restriction, including without limitation the rights to
Expand Down Expand Up @@ -230,12 +228,13 @@ private function action ($action_path, $http_method, $put_post_parameters = null
$action_response_array = json_decode($body_response, true);

$action_response_export = var_export($action_response_array, true);
error_log("----------------------------------------------\n" .
"FULL RESPONSE:\n\n{$action_response}\n\nEND FULL RESPONSE\n\n" .
"Headers:\n\n{$header_response}\n\nEnd Headers\n\n" .
"Data:\n\n{$body_response}\n\nEnd Data\n\n" .
"RESPONSE ARRAY:\n\n{$action_response_export}\n\nEND RESPONSE ARRAY\n" .
"----------------------------------------------");
// Per GH Issues #45/#47, commenting and may later expose this via option.
// error_log("----------------------------------------------\n" .
// "FULL RESPONSE:\n\n{$action_response}\n\nEND FULL RESPONSE\n\n" .
// "Headers:\n\n{$header_response}\n\nEnd Headers\n\n" .
// "Data:\n\n{$body_response}\n\nEnd Data\n\n" .
// "RESPONSE ARRAY:\n\n{$action_response_export}\n\nEND RESPONSE ARRAY\n" .
// "----------------------------------------------");

unset($action_response);
unset($action_response_export);
Expand Down

0 comments on commit d8f744a

Please sign in to comment.