Skip to content

fix: Keep source/binary compatibility with 2.5.1 for addRequestHeader#2022

Merged
johanandren merged 1 commit intomainfrom
wip-default-add-request-header-impls
Jan 31, 2025
Merged

fix: Keep source/binary compatibility with 2.5.1 for addRequestHeader#2022
johanandren merged 1 commit intomainfrom
wip-default-add-request-header-impls

Conversation

@johanandren
Copy link
Copy Markdown
Contributor

References #2021

@johanandren
Copy link
Copy Markdown
Contributor Author

Verified that this solves the issue for Scala, a project on akka-grpc-2.4.4 that gets a bump of the akka-grpc-runtime to a snapshot version of this PR works fine whie 2.5.2 causes the compilation error:

[error] Missing implementation for member of trait AkkaGrpcClient:
[error]   def addRequestHeader(key: String, value: String): akka.grpc.scaladsl.AkkaGrpcClient = ???

Need to check Java/mvn as well though.

@johanandren
Copy link
Copy Markdown
Contributor Author

Java verified, works fine 👍

@johanandren johanandren marked this pull request as ready for review January 31, 2025 15:19
@johanandren johanandren requested a review from efgpinto January 31, 2025 15:19
@johanandren johanandren merged commit 0034695 into main Jan 31, 2025
@johanandren johanandren deleted the wip-default-add-request-header-impls branch January 31, 2025 15:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants