Background

I am working on electron based application (an embedded express server in electron wrapper) and I am getting the blob url something like blob:localhost:3000/ ...

I just saw the code and looks like the createUrl method inside the coverUrl does not have second parameter and it defaults to blob always but I need base64 so I passed { replacements: "base64" } and a little change to recipe got it working ...

Problem

coverUrl() {
   return this.loaded.cover.then(() => {
    if (!this.cover) {
      return null;
    }
    if (this.archived) {
      return this.archive.createUrl(this.cover);
    } else {
      return this.cover;
    }
  });
}

Solution

Is there any issue passing the 2nd argument containing base64 flag ? like in the other versions ?
something like :

coverUrl() {
   return this.loaded.cover.then(() => {
    if (!this.cover) {
      return null;
    }
    if (this.archived) {
      return this.archive.createUrl(this.cover, {"base64": this.settings.replacements === "base64"});
    } else {
      return this.cover;
    }
  });
}

Note: There are some indentation changes showing up and I think Test cases might also have to be updated but I didn't ...
And I don't have any experience of contributing so if it fits in then good .. if not then Excuse me .. :)