Details
Improvement
Critical
Description
Multicraft updated their admin UI and API to add some new commands.
It doesn't appear we will need to make use of any of the new API commands, however, the API key authentication has changed to an HMAC rather than md5.
Multicraft's "MulticraftAPI.php" file will need to replace /multicraft/apis/multicraft_api.php. However, references to the CJSON class written in this file should be removed.
Looking over it, there may be a bug with the construction of the API $query parameters. If cURL is not enabled, a request will be made to:
$url . '?' . $query
...but $query begins with an ampersand (&), so you would end up with, for example:
url.com/?&key=value
Activity
Automated transition triggered when Cody Phillips (Inactive) merged pull request #5 in Stash -
| Status | In Review [ 5 ] | Closed [ 6 ] |
Tyson Phillips (Inactive)
made changes -
| Fix Version/s | 4.0.0 [ 10603 ] | |
| Fix Version/s | Short Term [ 10800 ] |
Automated transition triggered when Tyson Phillips (Inactive) created pull request #5 in Stash -
| Status | In Progress [ 3 ] | In Review [ 5 ] |
| Resolution | Fixed [ 1 ] |
Automated transition triggered when Tyson Phillips (Inactive) created a branch in Stash -
| Status | Open [ 1 ] | In Progress [ 3 ] |
Tyson Phillips (Inactive)
made changes -
| Description |
Multicraft updated their admin UI and API to add some new commands.
It doesn't appear we will need to make use of any of the new API commands, however, the API key authentication has changed to an HMAC rather than md5. Multicraft's "MulticraftAPI.php" file will need to replace /multicraft/apis/multicraft_api.php. However, references to the CJSON class written in this file should be removed. Looking over it, there may be a bug with the construction of the API $query parameters. If cURL is not enabled, a request will be made to: $url . '?' . $query ...but $query begins with an ampersand (&), so you would end up with: url.com/?&key=value |
Multicraft updated their admin UI and API to add some new commands.
It doesn't appear we will need to make use of any of the new API commands, however, the API key authentication has changed to an HMAC rather than md5. Multicraft's "MulticraftAPI.php" file will need to replace /multicraft/apis/multicraft_api.php. However, references to the CJSON class written in this file should be removed. Looking over it, there may be a bug with the construction of the API $query parameters. If cURL is not enabled, a request will be made to: $url . '?' . $query ...but $query begins with an ampersand (&), so you would end up with, for example: url.com/?&key=value |
Tyson Phillips (Inactive)
made changes -
| Field | Original Value | New Value |
|---|---|---|
| Attachment | MulticraftAPI.php [ 11442 ] |
Tyson Phillips (Inactive)
created issue -