Merge pull request #462 from brave/brave-stats-http-success

Fix stats updater to accept any 2XX HTTP response codes as success
brave · Sep 17, 2018 · 764a32a · 764a32a
2 parents f8f334c + 71d4a46
commit 764a32a
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/browser/brave_stats_updater.cc b/browser/brave_stats_updater.cc
@@ -103,7 +103,8 @@ void BraveStatsUpdater::OnSimpleLoaderComplete(
       simple_url_loader_->ResponseInfo()->headers)
     response_code =
         simple_url_loader_->ResponseInfo()->headers->response_code();
-  if (simple_url_loader_->NetError() != net::OK || response_code != 200) {
+  if (simple_url_loader_->NetError() != net::OK || response_code < 200 ||
+      response_code > 299) {
     LOG(ERROR) << "Failed to send usage stats to update server"
                << ", error: " << simple_url_loader_->NetError()
                << ", response code: " << response_code