Skip to content
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.

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

remote reset error with 1.8.8 and higher #434

Open
banool opened this issue Dec 5, 2023 · 2 comments
Open

remote reset error with 1.8.8 and higher #434

banool opened this issue Dec 5, 2023 · 2 comments

Comments

@banool
Copy link

banool commented Dec 5, 2023

This works:

docker run fullstorydev/grpcurl:v1.8.7 -d '{ "starting_version": 0 }'   -max-msg-sz 30000000   -H "authorization:Bearer aptoslabs_ep2o8nuDPuS_529dxW5UWEUY1YZeRWLM6EPCmsLNuz4BE"   grpc.mainnet.aptoslabs.com:443 aptos.indexer.v1.RawData/GetTransactions

You see data stream.

This does not:

docker run fullstorydev/grpcurl:v1.8.8 -d '{ "starting_version": 0 }'   -max-msg-sz 30000000   -H "authorization:Bearer aptoslabs_ep2o8nuDPuS_529dxW5UWEUY1YZeRWLM6EPCmsLNuz4BE"   grpc.mainnet.aptoslabs.com:443 aptos.indexer.v1.RawData/GetTransactions

You get this error instead:

Error invoking method "aptos.indexer.v1.RawData/GetTransactions": rpc error: code = Unavailable desc = failed to query for service descriptor "aptos.indexer.v1.RawData": upstream connect error or disconnect/reset before headers. reset reason: remote reset
@foo4u
Copy link

foo4u commented Jan 5, 2024

Just ran into this and found the issue. grpcurl 1.8.8 started looking for an endpoint called /grpc.reflection.v1.ServerReflection/ServerReflectionInfo before falling back to /grpc.reflection.v1alpha.ServerReflection/ServerReflectionInfo. However, it doesn't fall back on all error conditions.

@dgnemo
Copy link

dgnemo commented Sep 19, 2024

Hello @jhump, I am running into the same issue in my environment, except that my reverse-proxy is returning 401.

I see grpcreflect.NewClientAuto will fallback from v1 to v1alpha only in case of Unimplemented or Unavailable errors.

What do you think about doing a fallback for any error code, or at least supporting 401 too ?

Thanks!

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

No branches or pull requests

3 participants