QA » openQA Project

Observation¶

the autoinst logs do not render valid URL links.
The reason hides in the Anser.linkify. Which requires whitespaces between urls. The produced links also are incorrect when one is directly following another one.

[2024-09-12T08:54:16.658128Z] [info] [pid:37912] ::: OpenQA::Isotovideo::Utils::_fetch_new_refs: Updating Git cache for 'https://github.com/os-autoinst/os-autoinst-distri-openQA' under '/var/lib/openqa/cache/git//os-autoinst/os-autoinst-distri-openQA'

The quotes in this case end up as part of the linkification.

To reproduce¶

go to https://openqa.opensuse.org/tests/overview?distri=openqa&version=Tumbleweed&groupid=24
goto openqa_from_git autoinst-log.txt and check every link (git repos)
you will see something like

or

We noticed this in Clone.pm mainly
not sure how some URL conventions should be handled here or what improvement can be considered

Acceptance Criteria¶

• AC1: Only URLs from logs are linkified
• AC2: os-autoinst logs produce URLs that can reliably be linkified by the web UI

Suggestions¶

• Use double quotes consistently around URLs (or no quotes at all) and ensure that the URL is rendered correctly without the " character included in the link
  • the fact is double quote can't be in the URL (needs to be escaped everytime), single quotes can be in URL in unescaped form (however are usually escaped)
• Fix anser or use linkify or our own implementation
• Regarding links in autoinst-log.txt try out the rendering in terminal emulators