fix: Refer only required values from process.env instead of fetching …

…all (#933)

* Refer only required values from process.env instead of fetching all of them

* Update BaseTwilio.ts

* Fixing code style issues

* Update BaseTwilio.ts

---------

Co-authored-by: shrutiburman <[email protected]>

src/base/BaseTwilio.ts

@@ -75,23 +75,30 @@ namespace Twilio {
 
     constructor(username?: string, password?: string, opts?: ClientOpts) {
       this.opts = opts || {};
-      this.env = this.opts.env || process.env;
+      this.env = this.opts.env || {};
       this.username =
-        username ||
-        this.env.TWILIO_ACCOUNT_SID ||
+        username ??
+        this.env.TWILIO_ACCOUNT_SID ??
+        process.env.TWILIO_ACCOUNT_SID ??
         (() => {
           throw new Error("username is required");
         })();
       this.password =
-        password ||
-        this.env.TWILIO_AUTH_TOKEN ||
+        password ??
+        this.env.TWILIO_AUTH_TOKEN ??
+        process.env.TWILIO_AUTH_TOKEN ??
         (() => {
           throw new Error("password is required");
         })();
       this.accountSid = this.opts.accountSid || this.username;
-      this.edge = this.opts.edge || this.env.TWILIO_EDGE;
-      this.region = this.opts.region || this.env.TWILIO_REGION;
-      this.logLevel = this.opts.logLevel || this.env.TWILIO_LOG_LEVEL;
+      this.edge =
+        this.opts.edge ?? this.env.TWILIO_EDGE ?? process.env.TWILIO_EDGE;
+      this.region =
+        this.opts.region ?? this.env.TWILIO_REGION ?? process.env.TWILIO_REGION;
+      this.logLevel =
+        this.opts.logLevel ??
+        this.env.TWILIO_LOG_LEVEL ??
+        process.env.TWILIO_LOG_LEVEL;
       this.autoRetry = this.opts.autoRetry || false;
       this.maxRetries = this.opts.maxRetries;
       this.userAgentExtensions = this.opts.userAgentExtensions || [];