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

hcl2cdk: getExpressionAst fails for input string ''static'' #2913

Closed
sentry-io bot opened this issue Jun 1, 2023 · 3 comments · Fixed by #3018
Closed

hcl2cdk: getExpressionAst fails for input string ''static'' #2913

sentry-io bot opened this issue Jun 1, 2023 · 3 comments · Fixed by #3018
Assignees
Labels
bug Something isn't working feature/convert priority/important-longterm Medium priority, to be worked on within the following 1-2 business quarters. size/medium estimated < 1 week
Milestone

Comments

@sentry-io
Copy link

sentry-io bot commented Jun 1, 2023

Sentry Issue: CDKTF-REGISTRY-DOCS-VF

Error: getExpressionAst errors on input string ''static'': main.tf:0,0-1: Invalid character; Single quotes are not valid. Use double quotes (") to enclose strings., and 1 other diagnostic(s) 
  File "/__t/node/18.16.0/x64/lib/node_modules/cdktf-registry-docs/node_modules/@cdktf/hcl2json/lib/bridge.js", line 81, in cb
    err ? reject(new Error(err)) : resolve(...msg);
  File "/__t/node/18.16.0/x64/lib/node_modules/cdktf-registry-docs/node_modules/@cdktf/hcl2json/wasm/wasm_exec.js", line 366, in syscall/js.valueInvoke
    const result = Reflect.apply(v, undefined, args);
  File "wasm://wasm/018549f2:wasm-function[1489]", in <anonymous>
  File "wasm://wasm/018549f2:wasm-function[1473]", in <anonymous>
  File "wasm://wasm/018549f2:wasm-function[3125]", in <anonymous>
...
(5 additional frame(s) were not displayed)

This might be hard to debug since the snippet is filtered by sentry, would recommend looking at the breadcrumbs to find the right aws provider example.

@sentry-io
Copy link
Author

sentry-io bot commented Jun 1, 2023

Single quote makes it fail:
Sentry issue: CDKTF-REGISTRY-DOCS-PX

@DanielMSchmidt DanielMSchmidt added bug Something isn't working needs-priority Issue has not yet been prioritized; this will prompt team review feature/convert size/medium estimated < 1 week labels Jun 1, 2023
@DanielMSchmidt DanielMSchmidt added this to the 0.17 (tentative) milestone Jun 1, 2023
@mutahhir
Copy link
Member

@xiehan xiehan added priority/important-longterm Medium priority, to be worked on within the following 1-2 business quarters. and removed needs-priority Issue has not yet been prioritized; this will prompt team review labels Jul 20, 2023
@ansgarm ansgarm self-assigned this Jul 20, 2023
@github-actions
Copy link
Contributor

I'm going to lock this issue because it has been closed for 30 days. This helps our maintainers find and focus on the active issues. If you've found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 27, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working feature/convert priority/important-longterm Medium priority, to be worked on within the following 1-2 business quarters. size/medium estimated < 1 week
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants