Skip to content

Fixes Tinkoff/invest-api-java-sdk#61 change grpc-netty-shaded depende…#62

Merged
SRadyukov merged 3 commits intoTinkoff:mainfrom
zscauer:feature/native_compile_ready
Jul 10, 2023
Merged

Fixes Tinkoff/invest-api-java-sdk#61 change grpc-netty-shaded depende…#62
SRadyukov merged 3 commits intoTinkoff:mainfrom
zscauer:feature/native_compile_ready

Conversation

@zscauer
Copy link
Copy Markdown
Contributor

@zscauer zscauer commented May 26, 2023

Fixes #61
Change grpc-netty-shaded dependency to grpc-okhttp to get graalvm native compile compatibility

@SRadyukov
Copy link
Copy Markdown
Collaborator

we can break backward compatibility for some users here.
can we add dynamic support okhttp? for example if okhttp client present in classpath - initialize with it, instead use netty by default. for graal compiling you can exclude netty-shaded and add okhttp

@zscauer
Copy link
Copy Markdown
Contributor Author

zscauer commented Jun 14, 2023

исправил PR

@SRadyukov SRadyukov merged commit bbeafd5 into Tinkoff:main Jul 10, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Native image compatibility

3 participants