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.

Sign up for GitHub

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

Jump to bottom

Problem with number array parsing in Zeebe task handler #258

Closed
mvskvortsov opened this issue Sep 24, 2024 · 2 comments
Closed

Problem with number array parsing in Zeebe task handler #258

mvskvortsov opened this issue Sep 24, 2024 · 2 comments
Assignees
@jwulf

Comments

@mvskvortsov
Copy link

mvskvortsov commented Sep 24, 2024 •
edited
Loading

SDK Component

Zeebe

Expected Behavior

My input variables for ServiceTask is:
{"message":"Hello from automation","userId":null,"sendTo":[12022907,12022896,12022831]}

I need to get it in my task handler as they are.

Current Behavior

I get this in job.variables:
{"message":"Hello from automation","userId":null,"sendTo":[{"isLosslessNumber":true,"value":"12022907"},{"isLosslessNumber":true,"value":"12022896"},{"isLosslessNumber":true,"value":"12022831"}]}

Possible Solution

Steps to Reproduce

Context (Environment)
@jwulf
Copy link
Member

jwulf commented Sep 24, 2024

Sorry that this is not working as expected for you, and thank you for reporting it.

I have a good sense of what is going on here and how to fix it.

Let me take a look at it tomorrow and see if I can get a fix out for you.

@jwulf jwulf self-assigned this Sep 24, 2024
jwulf added a commit that referenced this issue Sep 24, 2024
@jwulf
fix(lossless-parser): correctly parse number array
0e1ae3d 
fixes #258
@mvskvortsov
Copy link
Author

@jwulf Thank you for your answer. When might a fix be available? Today I updated package to version 8.6.13 but the problem still exists.

@jwulf jwulf closed this as completed in d69729a Oct 24, 2024
github-actions bot pushed a commit that referenced this issue Oct 24, 2024
@semantic-release-bot
chore(release): 8.6.14 [skip ci]
1bbe026 
## [8.6.14](v8.6.13...v8.6.14) (2024-10-24)

### Bug Fixes

* **camunda8:** correctly parse autostart parameter of JobWorker ([cb95946](cb95946))
* **camunda8:** type variables in async process instance start as never ([3055734](3055734))
* **lossless-parser:** correctly parse number array ([d69729a](d69729a)), closes [#258](#258)
* **lossless-parser:** throw on encountering Date, Map, or Set ([bb5d8ea](bb5d8ea)), closes [#254](#254)
* **zeebe:** do not override explicit ZEEBE_GRPC_ADDRESS with default ZEEBE_ADDRESS ([cd6080f](cd6080f)), closes [#245](#245)
* **zeebe:** throw on client if array passed as variables to CompleteJob ([40a6316](40a6316)), closes [#247](#247)

### Features

* **camunda8:** add C8RestClient ([8e93c92](8e93c92)), closes [#235](#235)
* **camunda8:** add modifyAuthorization method ([0d97f68](0d97f68))
* **camunda8:** complete deployResources feature ([8043ac9](8043ac9))
* **camunda8:** implement createProcessInstanceWithResult ([4ec4fa1](4ec4fa1))
* **camunda8:** implement deleteResource over REST ([1dcb101](1dcb101)), closes [#251](#251)
* **camunda8:** implement deployResources REST API ([debd212](debd212))
* **camunda8:** implement publishMessage over REST ([057a9fe](057a9fe)), closes [#250](#250)
* **camunda8:** support broadcastSignal over REST ([43f82a4](43f82a4)), closes [#248](#248)
* **camunda8:** support pluggable winston logging for C8RestClient ([d41d3f8](d41d3f8))
* **camunda8:** support updateElementInstanceVariables ([7de82b7](7de82b7)), closes [#249](#249)
* **repo:** support passing middleware ([1b7715e](1b7715e)), closes [#261](#261)
* **zeebe:** add operationReference field to gRPC methods ([2e5af66](2e5af66)), closes [#237](#237)
* **zeebe:** create and cancel process instances over REST ([a49d217](a49d217))
* **zeebe:** lossless parse REST variables and customheaders ([f19a252](f19a252)), closes [#244](#244)
github-actions bot pushed a commit that referenced this issue Oct 24, 2024
@semantic-release-bot
chore(release): 8.6.14 [skip ci]
6192da7 
## [8.6.14](v8.6.13...v8.6.14) (2024-10-24)

### Bug Fixes

* **camunda8:** correctly parse autostart parameter of JobWorker ([cb95946](cb95946))
* **camunda8:** type variables in async process instance start as never ([3055734](3055734))
* **lossless-parser:** correctly parse number array ([d69729a](d69729a)), closes [#258](#258)
* **lossless-parser:** throw on encountering Date, Map, or Set ([bb5d8ea](bb5d8ea)), closes [#254](#254)
* **zeebe:** do not override explicit ZEEBE_GRPC_ADDRESS with default ZEEBE_ADDRESS ([cd6080f](cd6080f)), closes [#245](#245)
* **zeebe:** throw on client if array passed as variables to CompleteJob ([40a6316](40a6316)), closes [#247](#247)

### Features

* **camunda8:** add C8RestClient ([8e93c92](8e93c92)), closes [#235](#235)
* **camunda8:** add modifyAuthorization method ([0d97f68](0d97f68))
* **camunda8:** complete deployResources feature ([8043ac9](8043ac9))
* **camunda8:** implement createProcessInstanceWithResult ([4ec4fa1](4ec4fa1))
* **camunda8:** implement deleteResource over REST ([1dcb101](1dcb101)), closes [#251](#251)
* **camunda8:** implement deployResources REST API ([debd212](debd212))
* **camunda8:** implement publishMessage over REST ([057a9fe](057a9fe)), closes [#250](#250)
* **camunda8:** support broadcastSignal over REST ([43f82a4](43f82a4)), closes [#248](#248)
* **camunda8:** support pluggable winston logging for C8RestClient ([d41d3f8](d41d3f8))
* **camunda8:** support updateElementInstanceVariables ([7de82b7](7de82b7)), closes [#249](#249)
* **repo:** support passing middleware ([1b7715e](1b7715e)), closes [#261](#261)
* **zeebe:** add operationReference field to gRPC methods ([2e5af66](2e5af66)), closes [#237](#237)
* **zeebe:** create and cancel process instances over REST ([a49d217](a49d217))
* **zeebe:** lossless parse REST variables and customheaders ([f19a252](f19a252)), closes [#244](#244)
petar-slavov pushed a commit to petar-slavov/camunda-8-js-sdk that referenced this issue Mar 14, 2025
@semantic-release-bot @petar-slavov
chore(release): 8.6.14 [skip ci]
b9f73a8 
## [8.6.14](camunda/camunda-8-js-sdk@v8.6.13...v8.6.14) (2024-10-24)

### Bug Fixes

* **camunda8:** correctly parse autostart parameter of JobWorker ([cb95946](camunda@cb95946))
* **camunda8:** type variables in async process instance start as never ([3055734](camunda@3055734))
* **lossless-parser:** correctly parse number array ([d69729a](camunda@d69729a)), closes [camunda#258](camunda#258)
* **lossless-parser:** throw on encountering Date, Map, or Set ([bb5d8ea](camunda@bb5d8ea)), closes [camunda#254](camunda#254)
* **zeebe:** do not override explicit ZEEBE_GRPC_ADDRESS with default ZEEBE_ADDRESS ([cd6080f](camunda@cd6080f)), closes [camunda#245](camunda#245)
* **zeebe:** throw on client if array passed as variables to CompleteJob ([40a6316](camunda@40a6316)), closes [camunda#247](camunda#247)

### Features

* **camunda8:** add C8RestClient ([8e93c92](camunda@8e93c92)), closes [camunda#235](camunda#235)
* **camunda8:** add modifyAuthorization method ([0d97f68](camunda@0d97f68))
* **camunda8:** complete deployResources feature ([8043ac9](camunda@8043ac9))
* **camunda8:** implement createProcessInstanceWithResult ([4ec4fa1](camunda@4ec4fa1))
* **camunda8:** implement deleteResource over REST ([1dcb101](camunda@1dcb101)), closes [camunda#251](camunda#251)
* **camunda8:** implement deployResources REST API ([debd212](camunda@debd212))
* **camunda8:** implement publishMessage over REST ([057a9fe](camunda@057a9fe)), closes [camunda#250](camunda#250)
* **camunda8:** support broadcastSignal over REST ([43f82a4](camunda@43f82a4)), closes [camunda#248](camunda#248)
* **camunda8:** support pluggable winston logging for C8RestClient ([d41d3f8](camunda@d41d3f8))
* **camunda8:** support updateElementInstanceVariables ([7de82b7](camunda@7de82b7)), closes [camunda#249](camunda#249)
* **repo:** support passing middleware ([1b7715e](camunda@1b7715e)), closes [camunda#261](camunda#261)
* **zeebe:** add operationReference field to gRPC methods ([2e5af66](camunda@2e5af66)), closes [camunda#237](camunda#237)
* **zeebe:** create and cancel process instances over REST ([a49d217](camunda@a49d217))
* **zeebe:** lossless parse REST variables and customheaders ([f19a252](camunda@f19a252)), closes [camunda#244](camunda#244)
petar-slavov pushed a commit to petar-slavov/camunda-8-js-sdk that referenced this issue Mar 14, 2025
@semantic-release-bot @petar-slavov
chore(release): 8.6.14 [skip ci]
f53cd74 
## [8.6.14](camunda/camunda-8-js-sdk@v8.6.13...v8.6.14) (2024-10-24)

### Bug Fixes

* **camunda8:** correctly parse autostart parameter of JobWorker ([cb95946](camunda@cb95946))
* **camunda8:** type variables in async process instance start as never ([3055734](camunda@3055734))
* **lossless-parser:** correctly parse number array ([d69729a](camunda@d69729a)), closes [camunda#258](camunda#258)
* **lossless-parser:** throw on encountering Date, Map, or Set ([bb5d8ea](camunda@bb5d8ea)), closes [camunda#254](camunda#254)
* **zeebe:** do not override explicit ZEEBE_GRPC_ADDRESS with default ZEEBE_ADDRESS ([cd6080f](camunda@cd6080f)), closes [camunda#245](camunda#245)
* **zeebe:** throw on client if array passed as variables to CompleteJob ([40a6316](camunda@40a6316)), closes [camunda#247](camunda#247)

### Features

* **camunda8:** add C8RestClient ([8e93c92](camunda@8e93c92)), closes [camunda#235](camunda#235)
* **camunda8:** add modifyAuthorization method ([0d97f68](camunda@0d97f68))
* **camunda8:** complete deployResources feature ([8043ac9](camunda@8043ac9))
* **camunda8:** implement createProcessInstanceWithResult ([4ec4fa1](camunda@4ec4fa1))
* **camunda8:** implement deleteResource over REST ([1dcb101](camunda@1dcb101)), closes [camunda#251](camunda#251)
* **camunda8:** implement deployResources REST API ([debd212](camunda@debd212))
* **camunda8:** implement publishMessage over REST ([057a9fe](camunda@057a9fe)), closes [camunda#250](camunda#250)
* **camunda8:** support broadcastSignal over REST ([43f82a4](camunda@43f82a4)), closes [camunda#248](camunda#248)
* **camunda8:** support pluggable winston logging for C8RestClient ([d41d3f8](camunda@d41d3f8))
* **camunda8:** support updateElementInstanceVariables ([7de82b7](camunda@7de82b7)), closes [camunda#249](camunda#249)
* **repo:** support passing middleware ([1b7715e](camunda@1b7715e)), closes [camunda#261](camunda#261)
* **zeebe:** add operationReference field to gRPC methods ([2e5af66](camunda@2e5af66)), closes [camunda#237](camunda#237)
* **zeebe:** create and cancel process instances over REST ([a49d217](camunda@a49d217))
* **zeebe:** lossless parse REST variables and customheaders ([f19a252](camunda@f19a252)), closes [camunda#244](camunda#244)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jwulf jwulf

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@jwulf @mvskvortsov