Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Добавлена возможность отрезать лишную часть пути у issues #30

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

nluzgin
Copy link

@nluzgin nluzgin commented Oct 11, 2024

Новый параметр -pr; --path_replace

Пример эксплуатации:

... --path_replace=${CI_PROJECT_DIR}/

`Путь с рудиментами:

"fileName": "/builds/devsecops3000Pro/private/test-projects-and-other-trash/appsec.hub/genearal-test-pipe/code_scan/test_code_scan/PHP/.env.example",

Путь после применения параметра

code_scan/test_code_scan/PHP/.env.example`

vmpolosin and others added 3 commits October 9, 2024 16:59
Added replace path logic

See merge request devsecops3000Pro/private/appsec.hub/hub-tool-converters!1
@serpol1
Copy link
Collaborator

serpol1 commented Dec 18, 2024

Добрый день!

Общий входной параметр не очень подходит под такую задачу. Параметры у нас связаны с конверторами, форматами или параметрами сканирования.
Мы можем получить очень много лишних параметров для кастомизации вывода для всех случаев.

Предлагаю рассмотреть другие варианты.

С уважением,
Сергей Полшков

@nluzgin
Copy link
Author

nluzgin commented Dec 18, 2024

@serpol1 в целом позицию понимаю, но в таком случае ждём решения от вас.
Нас слишком длинный хвост в пути не устраивает. Коллеги теряются.
Если кому-то это будет ценно - мне кажется пусть оставляют как есть.
Также просто так понять что отрезать обычно не выйдет, пути разные, паттерны разные.

Нам такое нужно, мы такое УЖЕ используем. +1 параметр для нас не видится чем-то слишком диким. Хочешь кастомизацию - придётся чем-то платить.
На конкретно таком решении не настаиваем, но его альтернатива нам нужна.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants