Skip to content

Conversation

zhihuizhang17
Copy link
Contributor

No description provided.

@PragTob
Copy link

PragTob commented Sep 16, 2025

👋

We're not entirely sure but we experienced an endless loop in grpc_reflection which may be traced back to this (we didn't check deeply, but it seems to make sense). Anything we can do to help this move forward? We probably won't have enough time/priority to take it over but still offering maybe some smaller help :)

@mjheilmann
Copy link
Collaborator

@PragTob what version of the library were you running? We did an internal refactor since this PR was opened, plus this PR hasn't left draft so I'm afraid it may have been forgotten.

If you are on the latest version of grpc_reflection and saw the issue, I'd recommend opening an issue and provide info about your case, (if you are able) up to providing enough info to reproduce the issue.

@zhihuizhang17
Copy link
Contributor Author

zhihuizhang17 commented Sep 17, 2025

👋

We're not entirely sure but we experienced an endless loop in grpc_reflection which may be traced back to this (we didn't check deeply, but it seems to make sense). Anything we can do to help this move forward? We probably won't have enough time/priority to take it over but still offering maybe some smaller help :)

I noticed this issue several months ago, but I had a lot on my plate. I plan to continue working on it in two weeks, during the Chinese National Day Golden Week holiday.

@zhihuizhang17
Copy link
Contributor Author

@PragTob what version of the library were you running? We did an internal refactor since this PR was opened, plus this PR hasn't left draft so I'm afraid it may have been forgotten.

If you are on the latest version of grpc_reflection and saw the issue, I'd recommend opening an issue and provide info about your case, (if you are able) up to providing enough info to reproduce the issue.

Hi @mjheilmann , I created an issue for this.

@PragTob
Copy link

PragTob commented Oct 5, 2025

👋 We were on the latest released version 0.2.0 :)

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.

3 participants