Added back auth() and fixed begin-command to use new Firebase constructor

Author: ed7coyne

src/Firebase.cpp

@@ -37,6 +37,10 @@ Firebase::Firebase(const String& host, const String& auth) : host_(host), auth_(
   http_->setReuseConnection(true);
 }
 
+const String& Firebase::auth() const {
+  return auth_;
+}
+
 FirebaseGet Firebase::get(const String& path) {
   return FirebaseGet(host_, auth_, path, http_.get());
 }

src/Firebase.h

@@ -39,6 +39,8 @@ class Firebase {
  public:
   Firebase(const String& host, const String& auth = "");
 
+  const String& auth() const;
+
   // Fetch json encoded `value` at `path`.
   FirebaseGet get(const String& path);
   virtual std::unique_ptr<FirebaseGet> getPtr(const String& path);

src/modem/begin-command.cpp

@@ -33,10 +33,7 @@ bool BeginCommand::execute(const String& command,
     return false;
   }
 
-  new_firebase_.reset(new Firebase(host));
-  if (auth.length() != 0) {
-    new_firebase_->auth(auth);
-  }
+  new_firebase_.reset(new Firebase(host, auth));
 
   out->println("+OK");
   return true;

src/third-party/arduino-json-5.1.1/include/ArduinoJson/Arduino/String.hpp

@@ -9,8 +9,11 @@
 
 #ifndef ARDUINO
 
+#ifndef ARDUINO_STRING_OVERRIDE
+#define ARDUINO_STRING_OVERRIDE
 #include <string>
 typedef std::string String;
+#endif
 
 #else