addSingleton cannot support async function

[cuyl]

• Egg Version: 1.0.0

你好：
我想写个oracledb的插件，但是想给addSingleton传一个async函数，因为async function返回的是promise，所以如下的代码获取到的app.oracle就是一个Promise。

// app.js
const oracledb = require('oracledb');

module.exports = app => {
  app.addSingleton('oracle', createOnePool);
};

async function createOnePool(config, app) {
  return await oracledb.createPool(config);
}

我找到了源代码，发现我传入的函数在singleton.init()中执行
暂时我只能把这个init函数改成async函数来解决问题
lib/core/singleton.js#L19

async init() {
    const options = this.options;
    assert(!(options.client && options.clients),
      `eggg:singleton ${this.name} can not set options.client and options.clients both`);

    // alias app[name] as client, but still support createInstance method
    if (options.client) {
      const client = await this.createInstance(options.client); // await
      this.app[this.name] = client;
      assert(!client.createInstance, 'singleton instance should not have createInstance method');
      client.createInstance = this.createInstance.bind(this);
      return;
    }

    // multi clent, use app[name].getInstance(id)
    if (options.clients) {
      for (const id in options.clients) {
        this.clients.set(id, await this.createInstance(options.clients[id])); // await
      }
      this.app[this.name] = this;
      return;
    }

    // no config.clients and config.client
    this.app[this.name] = this;
  }

请问这个init函数能否改成async函数，如果不行是否有别的解决方案。

谢谢

[dead-horse]

addSingleton 设计的时候并没有考虑支持异步获取的形式，可以考虑不通过 addSingleton 来实现，而是在 app.js 中来实现这个功能:

// app.js

module.exports = app => {
  // 此处可以用 async function
  app.beforeStart(function* () {
    app.oracle = yield oracledb.createPool(app.config.oracle);
  });
}

[cuyl]

谢谢，看来最简单的方法就是这样了
参考了一下egg-mysql用的ali-rds，看来得在外面封装一层才能用addSingleton