Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Allow subprocess.Popen.stdout as file input #1055

Closed
@dvv

Description

@dvv

Steps to reproduce

@run_async
def pipe(bot, update):
  proc = subprocess.Popen(
    "cat image.jpg".split(),
    stdout=subprocess.PIPE
  )
  update.message.reply_voice(proc.stdout)
  while proc.returncode is None:
    proc.poll()

Expected behaviour

Allow subprocess.Popen.stdout be treated as valid io stream, just like open("image.jpg", "rb") is.

Actual behaviour

...
  File "/usr/lib/python3.6/site-packages/telegram/utils/request.py", line 262, in post
    data = InputFile(data)
  File "/usr/lib/python3.6/site-packages/telegram/files/inputfile.py", line 76, in __init__
    self.filename = os.path.basename(self.input_file.name)
  File "/usr/lib/python3.6/posixpath.py", line 144, in basename
    p = os.fspath(p)
TypeError: expected str, bytes or os.PathLike object, not int

Configuration

Operating System:

Linux xiaomi 4.15.9-1-ARCH #1 SMP PREEMPT Sun Mar 11 17:54:33 UTC 2018 x86_64 GNU/Linux

Version of Python, python-telegram-bot & dependencies:

Python 3.6.4 (default, Jan  5 2018, 02:35:40) 
[GCC 7.2.1 20171224] on linux

PROPOSED FIX

https://github.com/python-telegram-bot/python-telegram-bot/blob/master/telegram/files/inputfile.py#L76 should stringify self.input_file.name to look as:

self.filename = os.path.basename(str(self.input_file.name))

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions