embarklabs/embark
2
0
Fork 0
mirror of https://github.com/embarklabs/embark.git synced 2025-02-05 02:14:35 +00:00
Code Issues Projects Releases Wiki Activity

fix(@embark/ipfs): Update IPFS config CORS with default config

Browse Source 
Check if IPFS config has `API.HTTPHeaders.Access-Control-Allow-Origin` before attempting to update it.

`ipfs init` produces a default configuration without a `API.HTTPHeaders.Access-Control-Allow-Origin` element in the JSON. This caused an error to be thrown when attempting to update the IPFS config to provide CORS values.
This commit is contained in:
emizzle 2019-01-09 14:31:56 +11:00 committed by Pascal Precht
parent 05803fcf42
commit 518d319917
1 changed files with 12 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
src/lib/modules/ipfs/process.js
View File

@ -4,6 +4,8 @@ const constants = require('../../constants');
let ipfsProcess; // eslint-disable-line no-unused-vars
const IPFS_DEFAULT_CONFIG_ERROR = "API.HTTPHeaders key has no attributes";
class IPFSProcess extends ProcessWrapper {
constructor(options) {
super();
@ -64,12 +66,18 @@ class IPFSProcess extends ProcessWrapper {
// check cors config before updating if needed
self.getCorsConfig((err, config) => {
if(err){
if(err && err.indexOf(IPFS_DEFAULT_CONFIG_ERROR) === -1){
return console.error('Error getting IPFS CORS config: ', err);
}
let corsConfig = new Set(JSON.parse(config));
// test to ensure we have all cors needed
const needsUpdate = !self.cors.every(address => corsConfig.has(address));
let needsUpdate = false;
try {
let corsConfig = new Set(JSON.parse(config));
// test to ensure we have all cors needed
needsUpdate = !self.cors.every(address => corsConfig.has(address));
}
catch (_e) {
needsUpdate = true;
}
if(needsUpdate){
// update IPFS cors config
return self.updateCorsConfig(err => {