clean up file-related code that's not required anymore

Author: striezel

export-server/server.js

@@ -23,7 +23,6 @@ const http = require('http');
 const path = require('path');
 const ssr = require('./ssr.js');
 const url = require('url');
-const { v4: uuidv4 } = require('uuid');
 
 // Use hostname from environment variable HOST, if it is set.
 const hostname = process.env.HOST || 'localhost';
@@ -119,9 +118,8 @@ const server = http.createServer(function(req, res) {
     if (killed) {
       return;
     }
-    // Render file with JSDOM.
-    const filename = 'graph-' + uuidv4() + '.svg';
-    const result = await ssr.render(body, filename, req.headers["x-image-width"], req.headers["x-image-height"]);
+    // Render image with JSDOM.
+    const result = await ssr.render(body, req.headers["x-image-width"], req.headers["x-image-height"]);
     if (result.success) {
       res.statusCode = 200; // 200 == OK
       res.setHeader('Content-Type', 'image/svg+xml');

export-server/ssr.js

@@ -18,15 +18,13 @@
 
 var fs = require('fs');
 const jsdom = require('jsdom');
-const { v4: uuidv4 } = require('uuid');
 
 /* Renders JSON data for a Plotly.js plot into a SVG file.
 
    Parameters:
      jsonData - (string) the JSON data required by Plotly.js
-     filename - (string) desired output file name for the SVG file
-     width    - (number) width of the SVG file in pixels
-     height   - (number) height of the SVG file in pixels
+     width    - (number) width of the SVG image in pixels
+     height   - (number) height of the SVG image in pixels
 
    Returns:
      object that contains two members:
@@ -35,24 +33,13 @@ const { v4: uuidv4 } = require('uuid');
        failure - (string) reason for render failure; only present after failed
                  rendering, may be cryptic and is not necessarily human-friendly
 */
-exports.render = async function(jsonData, filename, width, height) {
-  const unique_id = uuidv4();
-
-  if (!filename) {
-    filename = 'graph-' + unique_id + '.svg';
-  }
+exports.render = async function(jsonData, width, height) {
   if (typeof jsonData !== 'string') {
     return {
       success: false,
       failure: 'json-not-a-string'
     };
   }
-  if (typeof filename !== 'string') {
-    return {
-      success: false,
-      failure: 'filename-not-a-string'
-    };
-  }
 
   const parsed_width = parseInt(width, 10);
   if (isNaN(parsed_width) || parsed_width <= 0) {