Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Timestamp and signature should be the last params in the order #193

Closed
wants to merge 1 commit into from

Conversation

Tharsanan1
Copy link

No description provided.

@@ -61,7 +61,7 @@ Likely causes:
Likely causes:
- Secret key is not set.
- Request was made with at least one empty parameter.
- `signature` is not the last parameter in the parameters list.
- `timestamp`&`signature` is not the last parameter in the parameters list.

Choose a reason for hiding this comment

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

The Likely causes: section refers to error message Signature for this request is not valid., which is concentrated on Signature, therefore shouldn't be related with the timestamp order.

Do you have a reproducible example we can check on to see the behavior? So that we can be sure of your use case.

Choose a reason for hiding this comment

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

The phrase is not grammatically correct. :)

@aisling-2
Copy link

Hi @Tharsanan1 , sorry for late reply, but just added some comments. Please check.

@2pd
Copy link
Contributor

2pd commented Mar 23, 2023

We found that it's a bug in this postman collection. The timestamp doesn't need to be in the second last position. It can be in any place except the last one.

Please update your collection with the latest release.

@2pd 2pd closed this Mar 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants