Skip to content
This repository has been archived by the owner on Nov 27, 2023. It is now read-only.

[BUG] Native build backend wipes out the expression content in verbose dialect #617

Closed
igarashitm opened this issue Apr 5, 2023 · 0 comments · Fixed by #621
Closed

[BUG] Native build backend wipes out the expression content in verbose dialect #617

igarashitm opened this issue Apr 5, 2023 · 0 comments · Fixed by #621
Assignees
Labels
bug Something isn't working

Comments

@igarashitm
Copy link
Contributor

igarashitm commented Apr 5, 2023

Describe the bug
While it's preserved with quarkus:dev, with native build, the expression content is wiped out with the /expression/simple/expression version of a dialect.

To Reproduce
Steps to reproduce the behavior:

  1. Paste this YAML
- from:
    uri: "timer:trigger-gitter-streaming"
    parameters:
        repeatCount: 1
    steps:
        - log:
              message: Opening connection to Gitter...
        - setHeader:
              name: Authorization
              expression:
                  simple:
                      expression: "Bearer {{token}}"
        - to:
              uri: >-
                  [https://gitter.ems.host/_matrix/client/v3/sync?filter={](https://gitter.ems.host/_matrix/client/v3/sync?filter=%7B)"account_data":{"not_types":["*"]},"room":{"account_data":{"not_types":["*"]},"rooms":["{{room}}"],"timeline":{"limit":1,"types":["m.room.message"]},"state":{"not_types":["*"]}
                  } }
        - log:
              message: Gitter HTTP Streaming started
  1. Click on 'Sync your code'
  2. With native build the expression is wiped out, while it's preserved with quarkus:dev

quarkus:dev

2023-04-05_12-20-16.mp4

native

2023-04-05_12-11-17.mp4

Expected behavior
Native build should also preserve it

Logs
If applicable, add logs to help explain your problem.

Environment (please complete the following information):

Additional context
Add any other context about the problem here.

@igarashitm igarashitm added the bug Something isn't working label Apr 5, 2023
@igarashitm igarashitm self-assigned this Apr 5, 2023
apupier added a commit to apupier/kaoto-backend that referenced this issue Apr 7, 2023
@apupier apupier self-assigned this Apr 7, 2023
apupier added a commit to apupier/kaoto-backend that referenced this issue Apr 7, 2023
fixes KaotoIO#617

Signed-off-by: Aurélien Pupier <[email protected]>
apupier added a commit to apupier/kaoto-backend that referenced this issue Apr 7, 2023
- it allows to have a non-regression test for KaotoIO#617 issue with
expressions
- extracted a single test from IntegrationsResourceTest which cannot be
launch in Native mode because it is using @Inject to access internal
services directly. it allows to inherit the native test class from
IntegrationsResourceTest and play all of the existing tests in native

part of KaotoIO#617
part of KaotoIO#437

Signed-off-by: Aurélien Pupier <[email protected]>
apupier added a commit to apupier/kaoto-backend that referenced this issue Apr 7, 2023
the Expression related classes were missing from native build;
Registered all of them for reflection. It might be possible to be more
specific and register a single one. to be investigated and potentially
improved in another iteration

fixes KaotoIO#617

Signed-off-by: Aurélien Pupier <[email protected]>
Delawen pushed a commit that referenced this issue Apr 10, 2023
- it allows to have a non-regression test for #617 issue with
expressions
- extracted a single test from IntegrationsResourceTest which cannot be
launch in Native mode because it is using @Inject to access internal
services directly. it allows to inherit the native test class from
IntegrationsResourceTest and play all of the existing tests in native

part of #617
part of #437

Signed-off-by: Aurélien Pupier <[email protected]>
Delawen pushed a commit that referenced this issue Apr 10, 2023
the Expression related classes were missing from native build;
Registered all of them for reflection. It might be possible to be more
specific and register a single one. to be investigated and potentially
improved in another iteration

fixes #617

Signed-off-by: Aurélien Pupier <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants