Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Avoid waiting for termination signal upon start-up #137

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

umairnadeem
Copy link

With the following plugin order in serverless.yml, we currently get stuck at the start-up page:

  - serverless-plugin-typescript
  - serverless-offline-kinesis
  - serverless-offline

package.json:

"serverless-offline": "^6.4.0",
"serverless-offline-kinesis": "^4.0.1",
"serverless-plugin-typescript": "^1.1.7"

@@ -75,8 +75,7 @@ class ServerlessOfflineKinesis {

async _startWithExplicitEnd() {
await this.start();
await this.ready();
this.end();
this.ready().then(this.end).catch(() => null);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@umairnadeem do you have some exemple of traces that calling t.end cause?

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants