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

Fix appsec:ruleset:update rake task #4205

Merged
merged 1 commit into from
Dec 9, 2024
Merged

Conversation

y9v
Copy link
Member

@y9v y9v commented Dec 9, 2024

What does this PR do?
This PR fixes error handling in appsec:ruleset:update rake task.

Motivation:
Exception when fetching the ruleset with invalid github access token.

Change log entry
None

Additional Notes:
None

How to test the change?
Run the task:

GITHUB_TOKEN=... bundle exec rake appsec:ruleset:update[1.13.3]

@y9v y9v self-assigned this Dec 9, 2024
@y9v y9v requested a review from a team as a code owner December 9, 2024 12:42
@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.77%. Comparing base (c07a8d4) to head (39bc695).

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #4205   +/-   ##
=======================================
  Coverage   97.77%   97.77%           
=======================================
  Files        1357     1357           
  Lines       81973    81974    +1     
  Branches     4168     4168           
=======================================
+ Hits        80145    80147    +2     
+ Misses       1828     1827    -1     

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

@pr-commenter
Copy link

pr-commenter bot commented Dec 9, 2024

Benchmarks

Benchmark execution time: 2024-12-09 13:04:13

Comparing candidate commit 39bc695 in PR branch appsec-fix-ruleset-update-task with baseline commit c07a8d4 in branch master.

Found 0 performance improvements and 0 performance regressions! Performance is the same for 31 metrics, 2 unstable metrics.

Copy link
Member

@Strech Strech left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍🏼

@y9v y9v merged commit 38fee9b into master Dec 9, 2024
346 checks passed
@y9v y9v deleted the appsec-fix-ruleset-update-task branch December 9, 2024 13:11
@github-actions github-actions bot added this to the 2.8.0 milestone Dec 9, 2024
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.

3 participants