Update license.py

[aylusltd]

Why are these changes needed?

Related issue number

Checks

• I've added tests (if relevant) corresponding to the changes introduced in this PR.
• I've made sure all auto checks have passed.

Summary by Sourcery

Bug Fixes:

• Add exception handling to the report function in license.py to prevent crashes when an error occurs during the HTTP request.

[sourcery-ai]

Reviewer's Guide by Sourcery

The PR updates the license reporting function by adding error handling and providing a default return value. The changes wrap the existing functionality in a try-except block and ensure the function always returns either a response object or False.

Sequence diagram for the updated license reporting function

sequenceDiagram
    participant User
    participant LicenseFunction as License Reporting Function
    participant RequestX as requestx

    User->>LicenseFunction: Call report(identifier, config, product)
    activate LicenseFunction

    alt config.reportId != 0 or config.shouldUpload
        LicenseFunction->>RequestX: POST request with data
        activate RequestX
        RequestX-->>LicenseFunction: response
        deactivate RequestX
        LicenseFunction-->>User: return response
    else
        LicenseFunction-->>User: return False
    end

    deactivate LicenseFunction

    Note over LicenseFunction: Wrapped in try-except block
    Note over LicenseFunction: Returns False on exception

Loading

File-Level Changes

Change	Details	Files
Added error handling to the license reporting function	Wrapped the existing code in a try-except block Added a return False statement for when conditions are not met Added a catch-all exception handler that returns False Maintained the same core functionality while making it more robust	src/verinfast/utils/license.py

---

Tips and commands

Interacting with Sourcery

• Trigger a new review: Comment @sourcery-ai review on the pull request.
• Continue discussions: Reply directly to Sourcery's review comments.
• Generate a GitHub issue from a review comment: Ask Sourcery to create an
  issue from a review comment by replying to it.
• Generate a pull request title: Write @sourcery-ai anywhere in the pull
  request title to generate a title at any time.
• Generate a pull request summary: Write @sourcery-ai summary anywhere in
  the pull request body to generate a PR summary at any time. You can also use
  this command to specify where the summary should be inserted.

Customizing Your Experience

Access your dashboard to:

• Enable or disable review features such as the Sourcery-generated pull request
  summary, the reviewer's guide, and others.
• Change the review language.
• Add, remove or edit custom review instructions.
• Adjust other review settings.

Getting Help

• Contact our support team for questions or feedback.
• Visit our documentation for detailed guides and information.
• Keep in touch with the Sourcery team by following us on X/Twitter, LinkedIn or GitHub.

[sourcery-ai]

Hey @aylusltd - I've reviewed your changes - here's some feedback:

Overall Comments:

• Consider catching specific exceptions (like requests.RequestException) instead of using a bare except clause, and add logging to help with debugging when errors occur.

Here's what I looked at during the review

• 🟢 General issues: all looks good
• 🟢 Security: all looks good
• 🟢 Testing: all looks good
• 🟢 Complexity: all looks good
• 🟢 Documentation: all looks good

---

Sourcery is free for open source - if you like our reviews please consider sharing them ✨

• X
• Mastodon
• LinkedIn
• Facebook

_{Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.}

[sourcery-ai]

issue (code-quality): We've found these issues:

• Inline variable that is immediately returned (inline-immediately-returned-variable)
• Remove unnecessary casts to int, str, float or bool (remove-unnecessary-cast)
• Use except Exception: rather than bare except: (do-not-use-bare-except)

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 77.77778% with 2 lines in your changes missing coverage. Please review.

Project coverage is 76.59%. Comparing base (e0656d4) to head (79af876).
Report is 3 commits behind head on main.

Files with missing lines	Patch %	Lines
src/verinfast/utils/license.py	77.77%	2 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #623      +/-   ##
==========================================
- Coverage   76.64%   76.59%   -0.05%     
==========================================
  Files          29       29              
  Lines        2137     2141       +4     
==========================================
+ Hits         1638     1640       +2     
- Misses        499      501       +2     

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

---

🚨 Try these New Features:

• Flaky Tests Detection - Detect and resolve failed and flaky tests
• JS Bundle Analysis - Avoid shipping oversized bundles