Skip to content
This repository was archived by the owner on Apr 3, 2024. It is now read-only.

fix reporting of file not found error#49

Merged
ofrobots merged 1 commit intogoogleapis:masterfrom
ofrobots:fix-file-not-found-error-reporting
Nov 30, 2015
Merged

fix reporting of file not found error#49
ofrobots merged 1 commit intogoogleapis:masterfrom
ofrobots:fix-file-not-found-error-reporting

Conversation

@ofrobots
Copy link
Copy Markdown
Contributor

The callback was getting called with the wrong value resulting the breakpoint
never being updated w/ the error on the server side. This causes the client to
wait upon a breakpoint forever even though we didn't even set the breakpoint in
the debuglet.

The callback was getting called with the wrong value resulting the breakpoint
never being updated w/ the error on the server side. This causes the client to
wait upon a breakpoint forever even though we didn't even set the breakpoint in
the debuglet.
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Nov 30, 2015
@ofrobots
Copy link
Copy Markdown
Contributor Author

@matthewloring ptal.

@matthewloring
Copy link
Copy Markdown
Contributor

Is there a way that the e2e tests could be expanded to make sure this path is tested somehow?

@matthewloring
Copy link
Copy Markdown
Contributor

Just saw #50, LGTM

@ofrobots ofrobots merged commit f016138 into googleapis:master Nov 30, 2015
@ofrobots ofrobots deleted the fix-file-not-found-error-reporting branch November 30, 2015 03:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants