Merge pull request #12875 from annando/simplify-oauth

Use the built in function to create a query string
This commit is contained in:
Philipp 2023-03-09 08:24:27 +01:00 committed by GitHub
commit 13c8e0c80e
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23

View file

@ -154,32 +154,9 @@ class OAuthUtil
public static function build_http_query($params) public static function build_http_query($params)
{ {
if (!$params) return '';
// Urlencode both keys and values
$keys = OAuthUtil::urlencode_rfc3986(array_keys($params));
$values = OAuthUtil::urlencode_rfc3986(array_values($params));
$params = array_combine($keys, $values);
// Parameters are sorted by name, using lexicographical byte value ordering. // Parameters are sorted by name, using lexicographical byte value ordering.
// Ref: Spec: 9.1.1 (1) // Ref: Spec: 9.1.1 (1)
uksort($params, 'strcmp'); uksort($params, 'strcmp');
return http_build_query($params, '', null, PHP_QUERY_RFC3986);
$pairs = [];
foreach ($params as $parameter => $value) {
if (is_array($value)) {
// If two or more parameters share the same name, they are sorted by their value
// Ref: Spec: 9.1.1 (1)
natsort($value);
foreach ($value as $duplicate_value) {
$pairs[] = $parameter . '=' . $duplicate_value;
}
} else {
$pairs[] = $parameter . '=' . $value;
}
}
// For each parameter, the name is separated from the corresponding value by an '=' character (ASCII code 61)
// Each name-value pair is separated by an '&' character (ASCII code 38)
return implode('&', $pairs);
} }
} }