Skip to content

Improve command logs errors#772

Merged
NicolasMahe merged 5 commits intodevfrom
feature/log-errors
Feb 21, 2019
Merged

Improve command logs errors#772
NicolasMahe merged 5 commits intodevfrom
feature/log-errors

Conversation

@NicolasMahe
Copy link
Copy Markdown
Member

@NicolasMahe NicolasMahe commented Feb 4, 2019

Dependant of #771

The command logs was not returning error from the stream and the command was not even terminated.
So any error on the logs (like the service exists but doesn't run) was not displaying an nothing show up.

The err channel was not returned to the user and just discarded:

go func() {	
  <-errC	
  closer()	
}()

I also add to change something in chunker: https://github.com/mesg-foundation/core/pull/772/files#diff-f59c295c71a834c1b8615304341180e4
The function Close was blocking..

krhubert
krhubert previously approved these changes Feb 5, 2019
@antho1404
Copy link
Copy Markdown
Member

Agree to merge this one when #771 is validated

@NicolasMahe NicolasMahe requested a review from krhubert February 7, 2019 11:46
@NicolasMahe NicolasMahe merged commit 73fe75a into dev Feb 21, 2019
@NicolasMahe NicolasMahe deleted the feature/log-errors branch February 21, 2019 09:13
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

Successfully merging this pull request may close these issues.

3 participants