ServerInterface: check_channel_request lacks message parameter
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 9.9k
- Forks
- 2.1k
- PR merge metrics
- No merged PRs in 30d
Description
paramiko.server.ServerInterface.check_global_request has msg parameter, so it's possible to decode any additional global request parameters:
def check_global_request(self, kind, msg):
return False
In contrast, paramiko.server.ServerInterface.check_channel_request lacks msg parameter:
def check_channel_request(self, kind, chanid):
return OPEN_FAILED_ADMINISTRATIVELY_PROHIBITED
Without msg, it's imposible to read additional open channel request parameters.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with paramiko.server.ServerInterface.check_channel_request and compare its signature with check_global_request. Trace where channel-open requests reach the server interface and determine how additional parameters are represented there. Done means channel request handlers can access those parameters, with coverage for the updated behavior.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- networking
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 42/100