Fix remote api logs

[ashley-cui]

properly returning logs through remote api
Closes: #7196
Closes: #6628

Signed-off-by: Ashley Cui [email protected]

[openshift-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: ashley-cui
To complete the pull request process, please assign mheon
You can assign the PR to them by writing /assign @mheon in a comment when ready.

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

• OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[zhangguanzhang]

show add some test like this

podman run --rm --tty busybox echo 1 | cat -A

and the bug is not belong to the remote

$ podman run --rm  alpine echo 1 | cat -A
1$
$ podman run --rm --tty alpine echo 1 | cat -A
1^M$
    [root@sz-test ~]# 

[ashley-cui]

@zhangguanzhang The logs wouldn't show because curl detected it as a binary, not application/json, so it didn't output anything.
I'm not sure what the cat -a is supposed to show, but using podman logs:

$ podman run --name hello  busybox echo 1 
1
$ podman logs hello
1

Feel free to test this PR and see if it fixes the issue

[zhangguanzhang]

must try this with the --tty

[ashley-cui]

@zhangguanzhang still works for me

$ podman run --tty --name hello  busybox echo 1 
1
$ podman logs hello
1

[zhangguanzhang]

with | cat -A

$ podman run --rm  alpine echo 1 | cat -A
1$
$ podman run --rm --tty alpine echo 1 | cat -A
1^M$
    [root@sz-test ~]# 

[ashley-cui]

@zhangguanzhang I'm not sure what the cat -A has to do with the remote podman logs command, since podman logs works.

[zhangguanzhang]

In this case, I may need to open a new issue

[ashley-cui]

@zhangguanzhang It might be worthwhile, since this PR is dealing with remote log output, not cli as you have been looking at

[ashley-cui]

uh oh looks like i broke something while fixing something..