jclouds/jclouds — pull request #1271 by richardcloudsoft (2013-01-31 14:10:41+00:00)
Title: Fix CloudStack URL signing for fields with [ chars

Commit 69a8304 caused the CloudStack QuerySigner to generate invalid
signatures where key names contained square brackets, such as in the
"iptonetworklist[N]" field to deployVirtualMachine. The commit changed
the whole query string being URL-encoded, whereas previously the field
values were encoded but the field names were not. The CloudStack API
guide says that values that should be encoded for signing but not field
names, and indeed the commit does cause signatures to be rejected.

This commit reverses the change to QuerySigner.createStringToSign() and
adds a unit test for this case.

This is the equivalent of pull request 1265 (which was on 1.5.x branch), applied to master.

Tests: unit tests (pass), live tests (no regressions)