Skip to content

feat(checker): add new checker for less#5767

Open
Anushkarshetty45 wants to merge 3 commits into
ossf:mainfrom
Anushkarshetty45:feat/checker-less
Open

feat(checker): add new checker for less#5767
Anushkarshetty45 wants to merge 3 commits into
ossf:mainfrom
Anushkarshetty45:feat/checker-less

Conversation

@Anushkarshetty45

Copy link
Copy Markdown

Summary:
Adds a new checker for GNU less ,a standard terminal pager
available in Alma Linux BaseOS packages.

Patterns found from binary analysis:

  • CONTAINS: Copyright string and PROGRAM identifier
  • VERSION: Extracted from "less (POSIX regular expressions)"
  • VENDOR_PRODUCT: ("gnu", "less")

Related:
Closes part of #2761 (Alma Linux package checker coverage)

Checklist:

  • isort passed
  • black passed
  • flake8 passed
  • bandit passed
  • codespell passed
  • pyupgrade passed

Signed-off-by: ANUSHKA R SHETTY <anushkarshetty45@gmail.com>
@ffontaine

Copy link
Copy Markdown
Collaborator

Tests must be added (including tests using "real" binaries). Documentation can be found here: https://cve-bin-tool.readthedocs.io/en/latest/checkers/README.html#adding-tests. You can also check what is done in other packages (e.g. d4b71df)

- Add less to checkers __init__.py __all__ list
- Add test/test_data/less.py with mapping and package test data
- Add filename mapping test in test_checkers.py

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Signed-off-by: ANUSHKA R SHETTY <anushkarshetty45@gmail.com>
@Anushkarshetty45

Copy link
Copy Markdown
Author

@ffontaine Added tests as requested ,test/test_data/less.py with mapping and package test data, registered the checker in init.py, and added the filename mapping test in test_checkers.py.

@ffontaine

Copy link
Copy Markdown
Collaborator

CI/CD tests failed because condensend downloads are missing from PR

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