Skip to content

Commit

Permalink
[#889] fixed that protocol headers was not merged with calculated hea…
Browse files Browse the repository at this point in the history
…ders e.g. containing authentication information

Signed-off-by: Thomas Jaeckle <thomas.jaeckle@bosch.io>
  • Loading branch information
thjaeckle committed Nov 30, 2020
1 parent dc4dc6c commit d0ee654
Showing 1 changed file with 4 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -249,9 +249,10 @@ private Optional<Signal<?>> jsonToDittoSignal(@Nullable final CloudEventData dat
.build();

final JsonifiableAdaptable jsonifiableAdaptable = ProtocolFactory.jsonifiableAdaptableFromJson(jsonObject);
return Optional.of(PROTOCOL_ADAPTER
.fromAdaptable(jsonifiableAdaptable)
.setDittoHeaders(adjustedHeaders));
final Signal<?> signal = PROTOCOL_ADAPTER.fromAdaptable(jsonifiableAdaptable);
final Signal<?> signalWithAdjustedHeaders = signal.setDittoHeaders(
signal.getDittoHeaders().toBuilder().putHeaders(adjustedHeaders).build());
return Optional.of(signalWithAdjustedHeaders);
}

private void ensureDataContentType(@Nullable final String dataContentType,
Expand Down

0 comments on commit d0ee654

Please sign in to comment.