[jira] [Resolved] (SSHD-534) SSHClient Channel id ( channel.getId() ) always returned as 0 , no matter how many channels are created.

["Goldstein Lyor (JIRA)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/SSHD-534?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Goldstein Lyor resolved SSHD-534.
---------------------------------
    Resolution: Not A Problem
      Assignee: Goldstein Lyor

It's not clear how you reached this conclusion - can you send test code that shows this problem (see test example below). Please note that if you create channels on different *sessions*, then indeed they might have the same id. Specifically, if you create one channel per session, then indeed all of them will have the value 0. This however is totally in line with the SSH specification - the channel id is unique only within the *same session* - there is no cross-session uniqueness guarantee. Furthermore, if you read [RFC 4254|https://tools.ietf.org/html/rfc4254] you will not find any such requirement - it is even theoretically possible to re-use a channel id if that channel has been closed (it's not a good idea for other reasons, but not because of the RFC...). The channel identifier (sender/receiver) are similar to the ephemeral ports used for a TCP/IP connections - these ports can (and are re-used) for *new* connections - like in the case of tearing down and opening a new SSH channel.

Here is a test that *proves* that different ID(s) are assigned to different channels on the *same session* - you can try it out by checking out the 0.14 code and adding this to the _ClientTest_ class:
{code:java}

    @Test
    public void testCreateChannelByType() throws Exception {
        client.start();

        ClientSession session = client.connect(getCurrentTestName(), "localhost", port).await().getSession();
        try {
            session.addPasswordIdentity(getCurrentTestName());
            session.auth().verify();

            try {
                Collection<ClientChannel> channels = new LinkedList<>();
                try {
                    channels.add(session.createChannel(ClientChannel.CHANNEL_SUBSYSTEM, "sftp"));
                    channels.add(session.createChannel(ClientChannel.CHANNEL_EXEC, getCurrentTestName()));
                    channels.add(session.createChannel(ClientChannel.CHANNEL_SHELL, getClass().getSimpleName()));
                    
                    Set<Integer> ids = new HashSet<Integer>(channels.size());
                    for (ClientChannel c : channels) {
                        int id = ((AbstractChannel) c).getId();
                        assertTrue("Channel ID repeated: " + id, ids.add(Integer.valueOf(id)));
                    }
                } finally {
                    for (ClientChannel c : channels) {
                        c.close(true);
                    }
                }
            } finally {
                session.close(true);
            }
        } finally {
            client.stop();
        }
    }
{code}
If you place a debug breakpoint at the _assertTrue_ statement you will see that each channel was assigned a consecutive channel id number.

> SSHClient Channel id ( channel.getId() ) always returned as 0 , no matter how many channels are created.
> --------------------------------------------------------------------------------------------------------
>
>                 Key: SSHD-534
>                 URL: https://issues.apache.org/jira/browse/SSHD-534
>             Project: MINA SSHD
>          Issue Type: Bug
>    Affects Versions: 0.14.0
>            Reporter: Preshit Maheshwari
>            Assignee: Goldstein Lyor
>             Fix For: 1.0.0
>
>
> Using SSHClient , if we are creating more than one channels using same client. Channel id return is 0 for all channels.
> It can be due to below check-in :
> https://github.com/apache/mina-sshd/commit/b971c90ca49b100bc8f6c8936b65aea90933a7ec
> nextChannelId is changed from static  



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)