fix: disable timeout if not set

Some of our operations take a really long time, if we don't set a
timeout for `ky` we get the default of 10 seconds. This PR sets
the timeout to `false` if one is not explicitly passed which
disables it.

Nb. I had to add the default to `false` to every invocation. Looking
at the code it should be enough to do it in `src/lib/configure.js`
but it doesn't seem to be.

Author: achingbrain

src/add/index.js

@@ -31,7 +31,7 @@ module.exports = configure(({ ky }) => {
     if (options.preload !== null) searchParams.set('preload', options.preload)
 
     const res = await ky.post('add', {
-      timeout: options.timeout,
+      timeout: options.timeout || false,
       signal: options.signal,
       headers: options.headers,
       searchParams,

src/block/rm-async-iterator.js

@@ -24,7 +24,7 @@ module.exports = configure(({ ky }) => {
     })
 
     const res = await ky.post('block/rm', {
-      timeout: options.timeout,
+      timeout: options.timeout || false,
       signal: options.signal,
       headers: options.headers,
       searchParams

src/cat.js

@@ -21,7 +21,7 @@ module.exports = configure(({ ky }) => {
     if (options.length) searchParams.set('length', options.length)
 
     const res = await ky.get('cat', {
-      timeout: options.timeout,
+      timeout: options.timeout || false,
       signal: options.signal,
       headers: options.headers,
       searchParams

src/config/profiles/apply.js

@@ -8,7 +8,7 @@ module.exports = configure(({ ky }) => {
     options = options || {}
 
     const res = await ky.post('config/profile/apply', {
-      timeout: options.timeout,
+      timeout: options.timeout || false,
       signal: options.signal,
       headers: options.headers,
       searchParams: {

src/config/profiles/list.js

@@ -9,7 +9,7 @@ module.exports = configure(({ ky }) => {
     options = options || {}
 
     const res = await ky.get('config/profile/list', {
-      timeout: options.timeout,
+      timeout: options.timeout || false,
       signal: options.signal,
       headers: options.headers
     })

src/files-regular/get.js

@@ -37,7 +37,7 @@ module.exports = configure(({ ky }) => {
     }
 
     const res = await ky.get('get', {
-      timeout: options.timeout,
+      timeout: options.timeout || false,
       signal: options.signal,
       headers: options.headers,
       searchParams

src/files-regular/ls.js

@@ -32,7 +32,7 @@ module.exports = configure(({ ky }) => {
     }
 
     const res = await ky.get('ls', {
-      timeout: options.timeout,
+      timeout: options.timeout || false,
       signal: options.signal,
       headers: options.headers,
       searchParams

src/files-regular/refs-local.js

@@ -10,7 +10,7 @@ module.exports = configure(({ ky }) => {
     options = options || {}
 
     const res = await ky.get('refs/local', {
-      timeout: options.timeout,
+      timeout: options.timeout || false,
       signal: options.signal,
       headers: options.headers
     })

src/files-regular/refs.js

@@ -50,7 +50,7 @@ module.exports = configure(({ ky }) => {
     }
 
     const res = await ky.get('refs', {
-      timeout: options.timeout,
+      timeout: options.timeout || false,
       signal: options.signal,
       headers: options.headers,
       searchParams

src/lib/configure.js

@@ -27,7 +27,7 @@ module.exports = create => config => {
     // https://github.com/sindresorhus/ky/pull/153
     ky: ky.extend({
       prefixUrl: config.apiAddr + config.apiPath,
-      timeout: config.timeout || 60 * 1000,
+      timeout: config.timeout || false,
       headers: config.headers,
       hooks: {
         afterResponse: [errorHandler]

src/pubsub/ls.js

@@ -7,7 +7,7 @@ module.exports = configure(({ ky }) => {
     options = options || {}
 
     const { Strings } = await ky.get('pubsub/ls', {
-      timeout: options.timeout,
+      timeout: options.timeout || false,
       signal: options.signal,
       headers: options.headers,
       searchParams: options.searchParams

src/pubsub/peers.js

@@ -15,7 +15,7 @@ module.exports = configure(({ ky }) => {
     searchParams.set('arg', topic)
 
     const { Strings } = await ky.get('pubsub/peers', {
-      timeout: options.timeout,
+      timeout: options.timeout || false,
       signal: options.signal,
       headers: options.headers,
       searchParams

src/pubsub/publish.js

@@ -12,7 +12,7 @@ module.exports = configure(({ ky }) => {
     searchParams.set('arg', topic)
 
     const res = await ky.post(`pubsub/pub?${searchParams}&arg=${encodeBuffer(data)}`, {
-      timeout: options.timeout,
+      timeout: options.timeout || false,
       signal: options.signal,
       headers: options.headers
     }).text()

src/pubsub/subscribe.js

@@ -38,7 +38,7 @@ module.exports = configure((config) => {
 
     try {
       res = await ky.post('pubsub/sub', {
-        timeout: options.timeout,
+        timeout: options.timeout || false,
         signal: options.signal,
         headers: options.headers,
         searchParams