Project

General

Profile

Actions

action #108554

open

[ux][ui] Wrong formatting on some links in single quotes includes trailing quotes to the link

Added by VANASTASIADIS over 2 years ago. Updated about 2 months ago.

Status:
New
Priority:
Low
Assignee:
-
Category:
Feature requests
Target version:
Start date:
2022-03-18
Due date:
% Done:

0%

Estimated time:
Tags:

Description

Observation

When trying to use openqa-clone-custom-git-refspec with NEEDLES_GIT_REFSPEC, openqa is unable to use the needle's link. In the details of the failed job, in the spot where it mentions the problem, it includes that link inside quotes as information. However, clicking on the link to open it includes the trailing quote in the address, leading to a 404 response (which can be fixed by manually removing the trailing quote in the address bar).

An example of this can be observed here. Near the bottom of the page, there are 3 occurrences of the link 'https://gitlab.suse.de/VANASTASIADIS/os-autoinst-needles-sles/tree/imagemagick_needles'. All three lead to a 404 page, but removing the trailing quote leads to the correct page.

Steps to reproduce

  • Run openqa-clone-custom-git-refspec <valid pr link> <valid job-to-clone link> MAX_JOB_TIME=7593869 NEEDLES_GIT_REFSPEC=<valid needle branch link> _GROUP=0 _SKIP_POST_FAIL_HOOKS=1
  • Go to the resulting openqa link and wait until the job fails
  • Go near the bottom of the screen and click on the needle link

Problem

It seems to be a formatting problem, and it's probably not affecting the actual use of the link in the code, as the opening quote exists too and is not part of the link. It should however be ensured that this is the case.

Suggestion

  • ensure it is a pure formatting bug
  • locate corresponding code
  • fix it so trailing quotes are not part of the link

openqa-clone-custom-git-refspec MAX_JOB_TIME=7593869 NEEDLES_GIT_REFSPEC= _GROUP=0 _SKIP_POST_FAIL_HOOKS=1


Related issues 1 (0 open1 closed)

Related to openQA Project - action #166676: Render valid URL links in WebUI autoinst-log size:SResolvedtinita2024-09-11

Actions
Actions #1

Updated by okurz over 2 years ago

  • Category set to Feature requests
  • Priority changed from Normal to Low
  • Target version set to future

Thanks for the report.

Actions #2

Updated by okurz over 2 years ago

  • Tags set to ux, ui
  • Subject changed from [tools] Wrong formatting on some links in single quotes includes trailing quotes to the link to [ux][ui] Wrong formatting on some links in single quotes includes trailing quotes to the link
Actions #3

Updated by okurz over 2 years ago

We might want to take a look how https://github.com/sindresorhus/linkify-urls solves that. Also maybe we can strip certain characters like ', ., , if they appear at the end of openQA urls as they are unlikely to appear in URLs we reference there

Actions #4

Updated by tinita about 2 months ago

Is this about the log file viewer?
This might have been solved in #166676 . @VANASTASIADIS can you confirm?
The test is already deleted, so I can't check.
It can help to add a bugref to a test's comments to make it not be deleted, btw.

Actions #5

Updated by tinita about 2 months ago

  • Related to action #166676: Render valid URL links in WebUI autoinst-log size:S added
Actions

Also available in: Atom PDF