Skip to content
This repository has been archived by the owner on Sep 16, 2024. It is now read-only.

fix(PE-5252): only certain params should be casted to numbers #88

Merged
merged 2 commits into from
Jan 8, 2024

Conversation

dtfiedler
Copy link
Collaborator

@dtfiedler dtfiedler commented Jan 8, 2024

Example request

http://api.arns.app/v1/contract/bLAgYxAdX2Ry-nt6aH2ixgvJXbpsEYm28NgJgyqfs-U/read/priceForInteraction?interactionName=extendRecord&name=666666&years=1

Current response:

Status: 400
Message:

Invalid input for interaction for extendRecord: name ('666666') must be string

Update with changes:

{
"contractTxId": "bLAgYxAdX2Ry-nt6aH2ixgvJXbpsEYm28NgJgyqfs-U",
"result": {
"input": {
"function": "priceForInteraction",
"interactionName": "extendRecord",
"name": "666666",
"years": 1
},
"price": 1000
},
"evaluationOptions": {
"maxInteractionEvaluationTimeSeconds": 3600
}
}

@dtfiedler dtfiedler marked this pull request as ready for review January 8, 2024 17:52
@dtfiedler dtfiedler requested a review from a team as a code owner January 8, 2024 17:52
@dtfiedler dtfiedler changed the title fix(price): only certain params should be casted to numbers fix(PE-5252): only certain params should be casted to numbers Jan 8, 2024
@dtfiedler dtfiedler merged commit 32ddb90 into main Jan 8, 2024
6 checks passed
@dtfiedler dtfiedler deleted the PE-5352-parse-query-params branch January 8, 2024 18:06
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants