string templated fields are not rendered literally

[YangMuye]

Apache Airflow version: 2.0.1

Kubernetes version (if you are using kubernetes) (use kubectl version): n/a

Environment:

• Cloud provider or hardware configuration:
• OS (e.g. from /etc/os-release): redhat 7.9
• Kernel (e.g. uname -a): 3.10.0-1160.11.1.el7.x86_64
• Install tools: pip
• Others:

What happened:

Given the following code

SSHOperator(task_id='dummy', command='''\
for i in 1..100
do
echo $i
echo $i $i
echo $i $i $i
done
''')

The command field is rendered as:

'\n\nfor i in 1..100\ndo\necho $i\necho $i $i\necho $i $i $i\ndone\n'

What you expected to happen:

I expect to see

for i in 1..100
do
echo $i
echo $i $i
echo $i $i $i
done

How to reproduce it:

Anything else we need to know:

This is because the following line renders templated fields using the pformat function.

airflow/airflow/www/views.py

Lines 926 to 928 in 35c9a90

	html_dict[template_field] = Markup("<pre><code>{}</pre></code>").format(
	pformat(content)
	) # noqa

I suggest we should render strings as is.

            if renderer in renderers:
                html_dict[template_field] = renderers[renderer](content)
            else:
                html_dict[template_field] = Markup("<pre><code>{}</pre></code>").format(
                     pformat(content)
+                   content if isinstance(content, str) else pformat(content)
                )  # noqa

[eladkal]

@YangMuye can you submit a PR with your suggest fix?

[jyotidhiman0610]

Let me know if you are working on this @YangMuye, if not I can take this up.

[YangMuye]

@jyotidhiman0610 Sorry, I am not working on it. Please feel free to take this up.

[uranusjr]

#15130 will fix this (by assigning the bash renderer to command).