Channel information is not appended to the return message in _recv_internal for PcanBus

[PetterEmanuelsson]

Describe the bug

I am experiencing issues with logging (at least two) Pcan Busses using Notifier and BLFWriter. I get a log file as expected but the issue lies in that the information regarding what channel the message logged inherited from is missing.

To Reproduce

Include any listener in a Notifier for multiple PcanBusses and try to extract the channel.
Please see below for example script to illustrate issue.

Expected behavior

Have a way of determining what channel the message originated from.
Please see below for example.

Additional context

OS and version: N/A
Python version: 3.13
python-can version: 4.5.0
python-can interface/s (if applicable): Pcan

Traceback and logs

Here's an example program that illustrates the issue. The Printer should have a way of showing the the channel the message originated from.

from can import Notifier, Printer
from can.interfaces.pcan import PcanBus
import time

bus_1 = PcanBus(channel='PCAN_USBBUS1', bitrate=500000)
bus_2 = PcanBus(channel='PCAN_USBBUS2', bitrate=250000)

listener = Printer()
notifier = Notifier([bus_1, bus_2], [listener])

while True:
    time.sleep(0.1)

Example solution:

In the _recv_internal function of PcanBus we can append the channel info to the rx_message

Original:

rx_msg = Message(
            timestamp=timestamp,
            arbitration_id=pcan_msg.ID,
            is_extended_id=is_extended_id,
            is_remote_frame=is_remote_frame,
            is_error_frame=is_error_frame,
            dlc=dlc,
            data=pcan_msg.DATA[:dlc],
            is_fd=is_fd,
            is_rx=is_rx,
            bitrate_switch=bitrate_switch,
            error_state_indicator=error_state_indicator,
        )

Example program output:

Timestamp: 1738764815.377878    ID:      5a0    S Rx                DL:  8    00 00 00 00 00 00 00 00
Timestamp: 1738764815.379947    ID:      702    S Rx                DL:  8    00 00 00 00 00 00 00 00
Timestamp: 1738764815.388107    ID:      5a0    S Rx                DL:  8    00 00 00 00 00 00 00 00
Timestamp: 1738764815.390177    ID:      702    S Rx                DL:  8    00 00 00 00 00 00 00 00

Changed:

rx_msg = Message(
            timestamp=timestamp,
            arbitration_id=pcan_msg.ID,
            is_extended_id=is_extended_id,
            is_remote_frame=is_remote_frame,
            is_error_frame=is_error_frame,
            dlc=dlc,
            data=pcan_msg.DATA[:dlc],
            is_fd=is_fd,
            is_rx=is_rx,
            bitrate_switch=bitrate_switch,
            error_state_indicator=error_state_indicator,
            channel=self.channel_info,
        )

Example program output:

Timestamp: 1738764985.982052    ID:      5a0    S Rx                DL:  8    00 00 00 00 00 00 00 00     Channel: PCAN_USBBUS1
Timestamp: 1738764985.984121    ID:      702    S Rx                DL:  8    00 00 00 00 00 00 00 00     Channel: PCAN_USBBUS1
Timestamp: 1738764985.985984    ID:      60a    S Rx                DL:  8    40 11 20 00 00 00 00 00     Channel: PCAN_USBBUS2
Timestamp: 1738764985.990200    ID:      58a    S Rx                DL:  8    4f 11 20 00 00 00 00 00     Channel: PCAN_USBBUS2

This enables a way of showing what channel the message originated from. Probably and ideally the channel should not be a string, rather an integer but I leave the complete solution here open since there are probably people here that has far more knowledge than me regarding this package and how this functionality can conform to how channel ids of other interfaces work.