search

binance / binance-api-postman

Postman collection for Binance Public API, including spot, margin, futures, etc.
MIT License
1.34k stars 893 forks source link

Added 'SPOT' to type on Daily Account Snapshot #176

Closed victorp13 closed 2 years ago

victorp13 commented 2 years ago

Since the Wallet\Daily Account Snapshot does not have a value set for "type=", it can end up being confusing for new users of your collection. One has to read the documentation to find out that setting a value for "type=" is mandatory.

In addition, I also encountered oddities with the Postman UI that lead to unexpected "Signature for this request is not valid." errors when manually setting the "type=" value manually in the URL entry field:

ss

As you can see above, the "type" parameter will be pushed to the bottom of the Params list, leading to the signature not being calculated correctly. This is because Postman does not send the parameters in order of the URL entry box, but in order of the Params list.

bnbot commented 2 years ago

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: victorp13

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files: - **[OWNERS](https://github.com/binance/binance-api-postman/blob/master/OWNERS)** Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
2pd commented 2 years ago
  1. type is provided in the postman collection. Please download the latest json file from here and import into your postman client. https://github.com/binance/binance-api-postman/blob/master/collections/binance_spot_api_v1.postman_collection.json#L12487
  2. Any parameter should not below the signature, otherwise it will be omitted during signature, but sent to API server.
  3. There is no way to review this PR, too many line of changes.