Fix default value of includeVariables parameter in tasklist api client #136

rshamborovskyy · 2024-04-19T16:34:21Z

Hello,

I am getting 500 error without specifying optional includeVariables, so I have to workaround it like so:

const taskVariables = await tasklist.getVariables({
  taskId,
  includeVariables: []
});

But anyway, here's the fix for that problem.

… search

jwulf · 2024-04-22T06:24:03Z

Thanks for this @rshamborovskyy

## [8.5.1-alpha.3](v8.5.1-alpha.2...v8.5.1-alpha.3) (2024-04-29) ### Bug Fixes * **tasklist:** correct default value of includeVariables parameter in tasklist variables search ([#136](#136)) ([23af921](23af921))

* feat(repo): add stack traces to async REST errors (#131) * feat(repo): add stack traces to async REST errors * test(operate): disable OAuth in unit test * test(operate): disable OAuth for Operate client in test * test(operate): rename integration test file * chore(repo): run unit test in pre-commit hook * test(operate): change process filename casing * chore(release): 8.5.1-alpha.1 [skip ci] ## [8.5.1-alpha.1](v8.5.0...v8.5.1-alpha.1) (2024-04-09) ### Features * **repo:** add stack traces to async REST errors ([#131](#131)) ([ef8d9c6](ef8d9c6)) * chore(release): 8.5.1-alpha.1 [skip ci] ## [8.5.1-alpha.1](v8.5.0...v8.5.1-alpha.1) (2024-04-09) ### Features * **repo:** add stack traces to async REST errors ([#131](#131)) ([ef8d9c6](ef8d9c6)) * docs(repo): add type of RESTError to JSDoc * docs(repo): update README.md * docs(repo): update README.md Fix Wrong Zeebe client method in readme file * chore(repo): ignore .idea folder in git To prevent IDE-specific settings from interfering with the project setup, the .idea folder, which is created by JetBrains IDEs, is now added to .gitignore. This ensures that developer-specific configurations do not pollute the project repository. * feat(repo): add status code to HTTPError type (#135) * feat(repo): add status code to HTTPError type Errors of type HTTPError now have a status code field * fix(oauth): retry once on 401 to get new token If a REST call fails with 401, we retry once in case it is a token expiry edge-case fixes #125 * test(operate): document the delay and eventual consistency measure in test * fix(oauth): decode jwt to get expiry time fixes #125 * chore(release): 8.5.1-alpha.2 [skip ci] ## [8.5.1-alpha.2](v8.5.1-alpha.1...v8.5.1-alpha.2) (2024-04-20) ### Features * **repo:** add status code to HTTPError type ([#135](#135)) ([cfea141](cfea141)), closes [#125](#125) [#125](#125) * chore(release): 8.5.1-alpha.2 [skip ci] ## [8.5.1-alpha.2](v8.5.1-alpha.1...v8.5.1-alpha.2) (2024-04-20) ### Features * **repo:** add status code to HTTPError type ([#135](#135)) ([cfea141](cfea141)), closes [#125](#125) [#125](#125) * ci(repo): update docker login command * ci(repo): update docker login command * ci: update docker login * build(repo): correct docker username * refactor(oauth): use got in place of node-fetch (#138) * refactor(oauth): use got in place of node-fetch * test(oauth): update cache eviction test * fix(tasklist): correct default value of includeVariables parameter in tasklist variables search (#136) * chore(release): 8.5.1-alpha.3 [skip ci] ## [8.5.1-alpha.3](v8.5.1-alpha.2...v8.5.1-alpha.3) (2024-04-29) ### Bug Fixes * **tasklist:** correct default value of includeVariables parameter in tasklist variables search ([#136](#136)) ([23af921](23af921)) * chore(release): 8.5.1-alpha.3 [skip ci] ## [8.5.1-alpha.3](v8.5.1-alpha.2...v8.5.1-alpha.3) (2024-04-29) ### Bug Fixes * **tasklist:** correct default value of includeVariables parameter in tasklist variables search ([#136](#136)) ([23af921](23af921)) * feat(repo): load system certs when custom cert specified * feat(repo): load system certs when custom cert specified fixes #139 * fix(oauth): throw if cacheDir not writeable on Windows * test(oauth): make cache dir read-only on Windows * test(oauth): make cache dir deleteable on Windows * test(oauth): make token cache dir read-only on Windows * test(oauth): hack around Windows test not throwing * refactor(repo): test self-signed certificate support fixes #139 fixes #141 * test(repo): isolate unit tests and do not run in integration envs * test(repo): run unit tests on Windows runner in CI * feat(zeebe): update gRPC package dep version * ci(repo): run unit tests on Windows for PRs * ci(repo): use cross-env to support Windows runner in CI * test(oauth): use execSync for Windows commands * refactor(repo): use win-ca for Windows system certificates * refactor(repo): refactor code to use win-ca on Windows * refactor(oauth): debug log in CI * revert(oauth): remove certificate debug statement * chore(release): 8.5.1-alpha.4 [skip ci] ## [8.5.1-alpha.4](v8.5.1-alpha.3...v8.5.1-alpha.4) (2024-05-03) ### Features * **repo:** load system certs when custom cert specified ([afce0a7](afce0a7)), closes [#139](#139) [#139](#139) [#141](#141) * chore(release): 8.5.1-alpha.4 [skip ci] ## [8.5.1-alpha.4](v8.5.1-alpha.3...v8.5.1-alpha.4) (2024-05-03) ### Features * **repo:** load system certs when custom cert specified ([afce0a7](afce0a7)), closes [#139](#139) [#139](#139) [#141](#141) --------- Co-authored-by: semantic-release-bot <semantic-release-bot@martynus.net> Co-authored-by: Hasan Alnatour <hassanalnator@gmail.com> Co-authored-by: Roman Shamborovskyy <shamborovskyy@gmail.com>

## [8.5.1](v8.5.0...v8.5.1) (2024-05-05) ### Features * **repo:** load system certs when custom cert specified ([#144](#144)) ([8a47d5e](8a47d5e)), closes [#131](#131) [#131](#131) [#131](#131) [#135](#135) [#125](#125) [#125](#125) [#125](#125) [#125](#125) [#125](#125) [#125](#125) [#138](#138) [#136](#136) [#136](#136) [#136](#136) [#139](#139) [#139](#139) [#141](#141) [#139](#139) [#139](#139) [#141](#141) [#139](#139) [#139](#139) [#141](#141)

Fix default value of includeVariables parameter in tasklist variables…

6eb33a6

… search

rshamborovskyy had a problem deploying to selfhosted April 22, 2024 06:23 — with GitHub Actions Failure

rshamborovskyy had a problem deploying to integration April 22, 2024 06:23 — with GitHub Actions Failure

jwulf self-assigned this Apr 22, 2024

rshamborovskyy had a problem deploying to selfhosted April 22, 2024 06:32 — with GitHub Actions Failure

rshamborovskyy had a problem deploying to integration April 22, 2024 06:32 — with GitHub Actions Failure

rshamborovskyy had a problem deploying to selfhosted April 22, 2024 06:32 — with GitHub Actions Failure

rshamborovskyy had a problem deploying to integration April 22, 2024 08:29 — with GitHub Actions Failure

rshamborovskyy had a problem deploying to selfhosted April 22, 2024 08:29 — with GitHub Actions Failure

jwulf merged commit 23af921 into camunda:alpha Apr 29, 2024
4 of 8 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Fix default value of includeVariables parameter in tasklist api client #136

Fix default value of includeVariables parameter in tasklist api client #136

rshamborovskyy commented Apr 19, 2024

jwulf commented Apr 22, 2024

Fix default value of includeVariables parameter in tasklist api client #136

Fix default value of includeVariables parameter in tasklist api client #136

Conversation

rshamborovskyy commented Apr 19, 2024

jwulf commented Apr 22, 2024