Merge pull request #152 from ESP32Async/fix

Urgent fix of previous PR #148: Arduino Core 2 has a different implementation of size()

Author: mathieucarbou

src/WebResponses.cpp

@@ -822,7 +822,7 @@ AsyncResponseStream::AsyncResponseStream(const char *contentType, size_t bufferS
   _contentType = contentType;
   // internal buffer will be null on allocation failure
   _content = std::unique_ptr<cbuf>(new cbuf(bufferSize));
-  if (_content->size() != bufferSize) {
+  if (bufferSize && _content->size() < bufferSize) {
 #ifdef ESP32
     log_e("Failed to allocate");
 #endif
@@ -840,6 +840,14 @@ size_t AsyncResponseStream::write(const uint8_t *data, size_t len) {
   if (len > _content->room()) {
     size_t needed = len - _content->room();
     _content->resizeAdd(needed);
+    // log a warning if allocation failed, but do not return: keep writing the bytes we can
+    // with _content->write: if len is more than the available size in the buffer, only
+    // the available size will be written
+    if (len > _content->room()) {
+#ifdef ESP32
+      log_e("Failed to allocate");
+#endif
+    }
   }
   size_t written = _content->write((const char *)data, len);
   _contentLength += written;