Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Add log message HTTP request and response to AG server #204

Merged
merged 1 commit into from
Aug 18, 2024

Conversation

pnt325
Copy link
Contributor

@pnt325 pnt325 commented Jul 28, 2024

Log URL and response code message for HTTP request to AG server

@@ -112,18 +116,22 @@ bool AgApiClient::postToServer(String data) {
String uri =
"http://hw.airgradient.com/sensors/airgradient:" + ag->deviceId() +
"/measures";
logInfo("Post uri: " + uri);
logInfo("Post data: " + data);
// logInfo("Post uri: " + uri);
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove?

logInfo("Post uri: " + uri);
logInfo("Post data: " + data);
// logInfo("Post uri: " + uri);
// logInfo("Post data: " + data);
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

remove?

Copy link
Contributor Author

@pnt325 pnt325 Aug 12, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm replace by new log bellow

arduino/src/AgApiClient.cpp

Lines 132 to 133 in ed7b8df

logInfo(String("POST: ") + uri);
logInfo(String("Return code: ") + String(retCode));

@airgradienthq airgradienthq merged commit be150e1 into develop Aug 18, 2024
20 checks passed
@pnt325 pnt325 deleted the feature/add-http-request-to-ag-log branch August 19, 2024 07:48
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants