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

listBreakpoint querystring encoding was incorrect#188

Merged
ofrobots merged 1 commit intogoogleapis:masterfrom
ofrobots:fix-query-encoding
Dec 14, 2016
Merged

listBreakpoint querystring encoding was incorrect#188
ofrobots merged 1 commit intogoogleapis:masterfrom
ofrobots:fix-query-encoding

Conversation

@ofrobots
Copy link
Copy Markdown
Contributor

No description provided.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Dec 14, 2016
}
url += '?success_on_timeout=' + encodeURIComponent(true);

var url = API + '/debuggees/' + encodeURIComponent(that.debuggeeId_) +

This comment was marked as spam.

This comment was marked as spam.

Copy link
Copy Markdown
Contributor

@matthewloring matthewloring left a comment

Choose a reason for hiding this comment

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

It would be good to get test coverage for these query parameters. We have a bug open tracking this.

@ofrobots
Copy link
Copy Markdown
Contributor Author

Added a bug (internal) to track lack of test coverage for success_on_timeout.

@ofrobots ofrobots merged commit 4c37f32 into googleapis:master Dec 14, 2016
@ofrobots ofrobots deleted the fix-query-encoding branch December 14, 2016 01:11
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