buffer: expose btoa and atob as globals

Signed-off-by: James M Snell <[email protected]>

Author: jasnell

.eslintrc.js

@@ -329,5 +329,7 @@ module.exports = {
     TextDecoder: 'readable',
     queueMicrotask: 'readable',
     globalThis: 'readable',
+    btoa: 'readable',
+    atob: 'readable',
   },
 };

doc/api/globals.md

@@ -146,6 +146,20 @@ This variable may appear to be global but is not. See [`__dirname`][].
 
 This variable may appear to be global but is not. See [`__filename`][].
 
+## `atob(data)`
+<!-- YAML
+added: REPLACEME
+-->
+
+Global alias for [`buffer.atob()`][].
+
+## `btoa(data)`
+<!-- YAML
+added: REPLACEME
+-->
+
+Global alias for [`buffer.btoa()`][].
+
 ## `clearImmediate(immediateObject)`
 <!-- YAML
 added: v0.9.1
@@ -402,6 +416,8 @@ The object that acts as the namespace for all W3C
 [`URL`]: url.md#url_class_url
 [`__dirname`]: modules.md#modules_dirname
 [`__filename`]: modules.md#modules_filename
+[`buffer.atob()`]: buffer.html#buffer_buffer_atob_data
+[`buffer.btoa()`]: buffer.html#buffer_buffer_btoa_data
 [`clearImmediate`]: timers.md#timers_clearimmediate_immediate
 [`clearInterval`]: timers.md#timers_clearinterval_timeout
 [`clearTimeout`]: timers.md#timers_cleartimeout_timeout

lib/internal/bootstrap/node.js

@@ -42,6 +42,7 @@ const {
   FunctionPrototypeCall,
   JSONParse,
   ObjectDefineProperty,
+  ObjectDefineProperties,
   ObjectGetPrototypeOf,
   ObjectPreventExtensions,
   ObjectSetPrototypeOf,
@@ -400,19 +401,37 @@ function setupGlobalProxy() {
 }
 
 function setupBuffer() {
-  const { Buffer } = require('buffer');
+  const {
+    Buffer,
+    atob,
+    btoa,
+  } = require('buffer');
   const bufferBinding = internalBinding('buffer');
 
   // Only after this point can C++ use Buffer::New()
   bufferBinding.setBufferPrototype(Buffer.prototype);
   delete bufferBinding.setBufferPrototype;
   delete bufferBinding.zeroFill;
 
-  ObjectDefineProperty(global, 'Buffer', {
-    value: Buffer,
-    enumerable: false,
-    writable: true,
-    configurable: true
+  ObjectDefineProperties(global, {
+    'Buffer': {
+      value: Buffer,
+      enumerable: false,
+      writable: true,
+      configurable: true,
+    },
+    'atob': {
+      value: atob,
+      enumerable: false,
+      writable: true,
+      configurable: true,
+    },
+    'btoa': {
+      value: btoa,
+      enumerable: false,
+      writable: true,
+      configurable: true,
+    },
   });
 }
 

test/common/index.js

@@ -263,6 +263,8 @@ let knownGlobals = [
   setInterval,
   setTimeout,
   queueMicrotask,
+  atob,
+  btoa,
 ];
 
 // TODO(@jasnell): This check can be temporary. AbortController is

test/parallel/test-btoa-atob-global.js

@@ -0,0 +1,9 @@
+'use strict';
+
+require('../common');
+
+const { strictEqual } = require('assert');
+const buffer = require('buffer');
+
+strictEqual(globalThis.atob, buffer.atob);
+strictEqual(globalThis.btoa, buffer.btoa);