gRPC: support unregistering dynamically registered methods

Open
#602 0 comments 0 reactions 0 assignees View on GitHub

Maintainers usually reply within 1 day

Nobody has claimed this yet.

Assessment

Difficulty
5/5
Estimated time
Over a week
Newbie friendliness
35/100
Issue type
Feature
Clarity
Mostly clear
Activity status
Active
Tech stack
c

Research direction

Start by locating the gRPC method registration and HTTP dispatcher code, then trace how handler contexts and built-in health routes are owned. The issue calls for an unregister API or internal removal primitive that handles concurrent requests and outstanding streams. Done means adding coverage for unary and streaming removal, replacement after removal, health cleanup, and destruction with active streams.

Written by the indexing model from the issue text.

Description

enhancement

Summary

The gRPC API supports adding and replacing methods but has no supported way to remove a method and its HTTP dispatcher.

Proposed improvement

Add an unregister API or an internal removal primitive with clear ownership rules for handler contexts, built-in health routes, and concurrent requests. Use the same primitive for transactional health registration and dynamic application configuration.

Tests

Cover unary and streaming removal, replacement after removal, health cleanup, and destruction with outstanding streams.

Dominant language
C
Stars
16
Forks
7
Avg merge
4h 28m
Merged PRs (30d)
191

Getting set up

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from c4punks/CWIST

All issues in c4punks/CWIST

Similar issues

More C issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.