raven-go icon indicating copy to clipboard operation
raven-go copied to clipboard

84 return errors on capturing failure

Open valentin-krasontovitsch opened this issue 7 years ago • 4 comments

Add methods that return error when capturing

The client's capture method returns an error channel that according to the docs is intended to be used for checking if a packet was sent successfully whenever that is important.

The ...AndWait methods use this channel, but only to wait. They do not capture the possible error coming from that channel.

The changes in this commit suggest to use the error and return it, so that a user may check whether a packet was sent successfully using new top level methods, instead of having to write their own.

The newly introduced methods follow the naming scheme Capture$SOMETHINGAndConfirm, where the returned error is considered confirmation.

Work on #84

valentin-krasontovitsch avatar Nov 20 '17 10:11 valentin-krasontovitsch

Any comments / change requests on this, @mattrobenolt? Have you had time to look at it yet?

valentin-krasontovitsch avatar Dec 06 '17 08:12 valentin-krasontovitsch

Bump.

valentin-krasontovitsch avatar Jan 22 '18 10:01 valentin-krasontovitsch

ping @mattrobenolt.

valentin-krasontovitsch avatar Feb 20 '18 10:02 valentin-krasontovitsch

why is there no reaction? is this repo dead? will you provide the possibility of someone else taking over, as the current developer(s) obviously don't use enough time to maintain the project? @mattrobenolt

don't mean to be cheeky, but getting a bit upset over being ignored...

valentin-krasontovitsch avatar Apr 04 '18 11:04 valentin-krasontovitsch