fix: appstore update crash and node 20 deprecation warnings - #162
Conversation
alaurie
commented
Aug 24, 2026
- Handle HttpException and IOException gracefully in appstore.java to skip catalogs returning HTTP 400 (e.g. malformed spaces in URLs) instead of retrying indefinitely and crashing the action.
- Update isSecondaryRateLimit to isRateLimit for broader rate limit detection.
- Update actions/checkout to v7.0.1 (3d3c42e) and jdx/mise-action to v4.2.5 (3c2e0cf) to resolve Node 20 deprecation warnings and run on Node 24. Pinned to commit hashes to maintain supply chain security policy.
- Handle HttpException and IOException gracefully in appstore.java to skip catalogs returning HTTP 400 (e.g. malformed spaces in URLs) instead of retrying indefinitely and crashing the action. - Update isSecondaryRateLimit to isRateLimit for broader rate limit detection. - Update actions/checkout to v7.0.1 (3d3c42e) and jdx/mise-action to v4.2.5 (3c2e0cf) to resolve Node 20 deprecation warnings and run on Node 24. Pinned to commit hashes to maintain supply chain security policy.
|
Important Review skippedAuto reviews are limited based on label configuration. 🏷️ Required labels (at least one) (1)
Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
0c107b3 to
08fb54f
Compare
|
Updated .github/appstore.java dependencies to their latest stable non-RC releases and updated deprecated GitHub API handler references:
|
|
the action runs to completion locally with no problem - worth a shot if this can help :) |