Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

[Proposal] add error event handler #155

Closed
Fhwang0926 opened this issue Jul 23, 2023 · 1 comment
Closed

[Proposal] add error event handler #155

Fhwang0926 opened this issue Jul 23, 2023 · 1 comment

Comments

@Fhwang0926
Copy link

There is currently no part of the error event processing Anyone else think they should add?

function getCause(isStream, res, json, callback) {
    var chunks = '';
    if (isStream) {
      res.on('data', function (chunk) {
        chunks += chunk;
      });
      res.on('end', function () {
        callback(null, utils.parseJSON(chunks) || chunks);
      });
    } else {
      callback(null, json);
    }
  }

During exec processing in the dockerode project, there was an issue where error event processing was absent when the container was restarted, so the exception was not processed

So I'm leaving my opinion here

@Fhwang0926 Fhwang0926 changed the title [] [Proposal] add error event handler Jul 23, 2023
@apocas
Copy link
Owner

apocas commented Oct 11, 2023

@apocas apocas closed this as completed Oct 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants