fix: remove http api support for key.export

ipfs · Aug 5, 2020 · 782235e · 782235e
1 parent 2f89e87
commit 782235e
Show file tree

Hide file tree

Showing 7 changed files with 0 additions and 159 deletions.
diff --git a/packages/interface-ipfs-core/src/key/export.js b/packages/interface-ipfs-core/src/key/export.js
diff --git a/packages/interface-ipfs-core/src/key/index.js b/packages/interface-ipfs-core/src/key/index.js
@@ -6,7 +6,6 @@ const tests = {
   list: require('./list'),
   rename: require('./rename'),
   rm: require('./rm'),
-  export: require('./export'),
   import: require('./import')
 }
 

diff --git a/packages/ipfs-http-client/src/key/export.js b/packages/ipfs-http-client/src/key/export.js
diff --git a/packages/ipfs-http-client/src/key/index.js b/packages/ipfs-http-client/src/key/index.js
@@ -5,6 +5,5 @@ module.exports = config => ({
   list: require('./list')(config),
   rename: require('./rename')(config),
   rm: require('./rm')(config),
-  export: require('./export')(config),
   import: require('./import')(config)
 })
diff --git a/packages/ipfs/src/http/api/resources/key.js b/packages/ipfs/src/http/api/resources/key.js
@@ -181,50 +181,6 @@ exports.gen = {
   }
 }
 
-exports.export = {
-  options: {
-    validate: {
-      options: {
-        allowUnknown: true,
-        stripUnknown: true
-      },
-      query: Joi.object().keys({
-        name: Joi.string().required(),
-        password: Joi.string().required(),
-        timeout: Joi.timeout()
-      })
-        .rename('arg', 'name', {
-          override: true,
-          ignoreUndefined: true
-        })
-    }
-  },
-  handler: async (request, h) => {
-    const {
-      app: {
-        signal
-      },
-      server: {
-        app: {
-          ipfs
-        }
-      },
-      query: {
-        name,
-        password,
-        timeout
-      }
-    } = request
-
-    const pem = await ipfs.key.export(name, password, {
-      signal,
-      timeout
-    })
-
-    return h.response(pem).type('application/x-pem-file')
-  }
-}
-
 exports.import = {
   options: {
     validate: {

diff --git a/packages/ipfs/src/http/api/routes/key.js b/packages/ipfs/src/http/api/routes/key.js
@@ -23,11 +23,6 @@ module.exports = [
     path: '/api/v0/key/rename',
     ...resources.key.rename
   },
-  {
-    method: 'POST',
-    path: '/api/v0/key/export',
-    ...resources.key.export
-  },
   {
     method: 'POST',
     path: '/api/v0/key/import',

diff --git a/packages/ipfs/test/http-api/inject/key.js b/packages/ipfs/test/http-api/inject/key.js
@@ -18,7 +18,6 @@ describe('/key', function () {
         rm: sinon.stub(),
         rename: sinon.stub(),
         gen: sinon.stub(),
-        export: sinon.stub(),
         import: sinon.stub()
       }
     }
@@ -228,50 +227,6 @@ describe('/key', function () {
     })
   })
 
-  describe('/export', () => {
-    const defaultOptions = {
-      signal: sinon.match.instanceOf(AbortSignal),
-      timeout: undefined
-    }
-
-    it('only accepts POST', () => {
-      return testHttpMethod('/api/v0/key/export')
-    })
-
-    it('should export a key', async () => {
-      const name = 'name'
-      const password = 'password'
-
-      ipfs.key.export.withArgs(name, password, defaultOptions).returns('pem')
-
-      const res = await http({
-        method: 'POST',
-        url: `/api/v0/key/export?arg=${name}&password=${password}`
-      }, { ipfs })
-
-      expect(res).to.have.property('statusCode', 200)
-      expect(res).to.have.property('result', 'pem')
-    })
-
-    it('accepts a timeout', async () => {
-      const name = 'name'
-      const password = 'password'
-
-      ipfs.key.export.withArgs(name, password, {
-        ...defaultOptions,
-        timeout: 1000
-      }).returns('pem')
-
-      const res = await http({
-        method: 'POST',
-        url: `/api/v0/key/export?arg=${name}&password=${password}&timeout=1s`
-      }, { ipfs })
-
-      expect(res).to.have.property('statusCode', 200)
-      expect(res).to.have.property('result', 'pem')
-    })
-  })
-
   describe('/import', () => {
     const defaultOptions = {
       signal: sinon.match.instanceOf(AbortSignal),