Skip to content

@Header("'foo.bar'") looks for a header ''foo' with property bar' #3083

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
Closed
garyrussell opened this issue Oct 16, 2019 · 0 comments · Fixed by #3084
Closed

@Header("'foo.bar'") looks for a header ''foo' with property bar' #3083

garyrussell opened this issue Oct 16, 2019 · 0 comments · Fixed by #3084
Assignees
@garyrussell
Milestone
5.2.1

Comments

@garyrussell
Copy link
Contributor

garyrussell commented Oct 16, 2019 •
edited
Loading

See https://stackoverflow.com/questions/58403246/header-annotation-with-in-header-name-such-as-headerfoo-bar-doesnt-m/58414528#58414528

When @Header("'foo.bar'") is used, we end up with:

#target.handleMessage(payload, #requiredHeader(headers, ''foo').bar')
@garyrussell garyrussell added this to the 5.2.1 milestone Oct 16, 2019
@garyrussell garyrussell changed the title @Header("foo.bar") looks for a header 'foo' with property bar @Header("'foo.bar'") looks for a header ''foo' with property bar' Oct 16, 2019
@garyrussell garyrussell self-assigned this Oct 16, 2019
garyrussell added a commit to garyrussell/spring-integration that referenced this issue Oct 16, 2019
@garyrussell
spring-projectsGH-3083: Support @Header with dotted literals
f83a608 
Fixes spring-projects#3083

`@Header("foo.bar"` means extract property `bar` from header `foo`)`

Support `@Header("'foo.bar'")`, meaning get the value of header `foo.bar`.

**cherry-pick to 5.1.x, 4.3.x**
@garyrussell garyrussell mentioned this issue Oct 16, 2019
Merged
garyrussell added a commit to garyrussell/spring-integration that referenced this issue Oct 16, 2019
@garyrussell
spring-projectsGH-3083: Support @Header with dotted literals
5ba036d 
Fixes spring-projects#3083

`@Header("foo.bar")` means extract property `bar` from header `foo`.

Support `@Header("'foo.bar'")`, meaning get the value of header `foo.bar`.

**cherry-pick to 5.1.x, 4.3.x**
artembilan pushed a commit that referenced this issue Oct 16, 2019
@garyrussell @artembilan
GH-3083: Support @Header with dotted literals
87af95e 
Fixes #3083

`@Header("foo.bar")` means extract property `bar` from header `foo`.

Support `@Header("'foo.bar'")`, meaning get the value of header `foo.bar`.

**cherry-pick to 5.1.x, 4.3.x**
artembilan pushed a commit that referenced this issue Oct 16, 2019
@garyrussell @artembilan
GH-3083: Support @Header with dotted literals
86f0ca2 
Fixes #3083

`@Header("foo.bar")` means extract property `bar` from header `foo`.

Support `@Header("'foo.bar'")`, meaning get the value of header `foo.bar`.

**cherry-pick to 5.1.x, 4.3.x**

# Conflicts:
#	spring-integration-core/src/test/java/org/springframework/integration/handler/MethodInvokingMessageProcessorTests.java
artembilan pushed a commit that referenced this issue Oct 16, 2019
@garyrussell @artembilan
GH-3083: Support @Header with dotted literals
9c24e16 
Fixes #3083

`@Header("foo.bar")` means extract property `bar` from header `foo`.

Support `@Header("'foo.bar'")`, meaning get the value of header `foo.bar`.

**cherry-pick to 5.1.x, 4.3.x**

# Conflicts:
#	spring-integration-core/src/test/java/org/springframework/integration/handler/MethodInvokingMessageProcessorTests.java

# Conflicts:
#	spring-integration-core/src/main/java/org/springframework/integration/handler/support/MessagingMethodInvokerHelper.java
#	spring-integration-core/src/test/java/org/springframework/integration/handler/MethodInvokingMessageProcessorTests.java
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@garyrussell garyrussell

Labels
None yet
Projects
None yet
Milestone
5.2.1
Development

Successfully merging a pull request may close this issue.

GH-3083: Support @Header with dotted literals garyrussell/spring-integration
1 participant
@garyrussell