Skip to content

Rubygems API Key & logging#823

Merged
babenek merged 11 commits intoSamsung:dependabot/pip/cryptography-46.0.6from
babenek:auxiliary
Mar 30, 2026
Merged

Rubygems API Key & logging#823
babenek merged 11 commits intoSamsung:dependabot/pip/cryptography-46.0.6from
babenek:auxiliary

Conversation

@babenek
Copy link
Copy Markdown
Contributor

@babenek babenek commented Mar 27, 2026

Description

Please include a summary of the change and which is fixed.

  • Add Rubygems API key pattern
  • Apply lazy logging
  • Reduce logging level error to warning in deep scanners

How has this been tested?

Please describe the tests that you ran to verify your changes.

  • UnitTest
  • Benchmark

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Mar 27, 2026

Codecov Report

❌ Patch coverage is 54.21687% with 38 lines in your changes missing coverage. Please review.
✅ Project coverage is 92.29%. Comparing base (0739b4c) to head (4930c00).

Files with missing lines Patch % Lines
credsweeper/file_handler/diff_content_provider.py 0.00% 4 Missing ⚠️
credsweeper/main.py 66.66% 3 Missing ⚠️
credsweeper/deep_scanner/eml_scanner.py 0.00% 2 Missing ⚠️
credsweeper/deep_scanner/pdf_scanner.py 0.00% 2 Missing ⚠️
credsweeper/deep_scanner/rpm_scanner.py 0.00% 2 Missing ⚠️
credsweeper/file_handler/file_path_extractor.py 50.00% 2 Missing ⚠️
credsweeper/ml_model/ml_validator.py 33.33% 2 Missing ⚠️
credsweeper/rules/rule.py 0.00% 2 Missing ⚠️
credsweeper/credentials/credential_manager.py 0.00% 1 Missing ⚠️
credsweeper/deep_scanner/abstract_scanner.py 66.66% 1 Missing ⚠️
... and 17 more
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #823      +/-   ##
==========================================
- Coverage   92.29%   92.29%   -0.01%     
==========================================
  Files         145      145              
  Lines        5905     5904       -1     
  Branches      967      966       -1     
==========================================
- Hits         5450     5449       -1     
  Misses        294      294              
  Partials      161      161              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@babenek babenek marked this pull request as ready for review March 27, 2026 09:54
@babenek babenek requested a review from a team as a code owner March 27, 2026 09:54
@babenek babenek changed the base branch from main to dependabot/pip/cryptography-46.0.6 March 30, 2026 09:43
@babenek babenek merged commit 51ac76c into Samsung:dependabot/pip/cryptography-46.0.6 Mar 30, 2026
33 checks passed
@babenek babenek deleted the auxiliary branch March 30, 2026 09:44
babenek added a commit that referenced this pull request Mar 30, 2026
* Bump cryptography from 46.0.5 to 46.0.6

Bumps [cryptography](https://github.com/pyca/cryptography) from 46.0.5 to 46.0.6.
- [Changelog](https://github.com/pyca/cryptography/blob/main/CHANGELOG.rst)
- [Commits](pyca/cryptography@46.0.5...46.0.6)

---
updated-dependencies:
- dependency-name: cryptography
  dependency-version: 46.0.6
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

* Rubygems API Key & logging (#823)

* rubygems

* warnings in deep scanners

* pylint cfg

* lazy logging

* pylin-fix

* error->warning

* step-security/harden-runner@fa2e9d6 # v2.16.0 - 2026.03.16

* fix ci warning

* Update PyPI publish action to version 1.13.0

* Update dependency-review-action to version 4.9.0

* Update cryptography package version to 46.0.6

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Roman Babenko <babenek@users.noreply.github.com>
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