chore: updated the RL wrapper installation path#746
Merged
sanchitmehtagit merged 1 commit intomainfrom Feb 19, 2026
Merged
Conversation
NandanPrabhu
approved these changes
Feb 19, 2026
There was a problem hiding this comment.
Pull request overview
Updates the GitHub composite action used to run the ReversingLabs (RL) scanner by changing where the rl-wrapper Python package is installed from.
Changes:
- Switches the
pip --index-urlpath used to installrl-wrapperfrom the internal Artifactory PyPI endpoint. - Removes the
rl-wrapperversion constraint during installation.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| shell: bash | ||
| run: | | ||
| pip install rl-wrapper>=1.0.0 --index-url "https://${{ env.PRODSEC_TOOLS_USER }}:${{ env.PRODSEC_TOOLS_TOKEN }}@a0us.jfrog.io/artifactory/api/pypi/python-local/simple" | ||
| pip install rl-wrapper --index-url "https://${{ env.PRODSEC_TOOLS_USER }}:${{ env.PRODSEC_TOOLS_TOKEN }}@a0us.jfrog.io/artifactory/api/pypi/python/simple" |
There was a problem hiding this comment.
The RL wrapper install command no longer specifies any version constraint. This makes the action non-reproducible and can introduce unexpected breakages if a new (or old) rl-wrapper release changes behavior. Consider restoring a lower bound (e.g., the previous ">=1.0.0") or pinning to a known-good version/range that matches the scanner expectations.
Suggested change
| pip install rl-wrapper --index-url "https://${{ env.PRODSEC_TOOLS_USER }}:${{ env.PRODSEC_TOOLS_TOKEN }}@a0us.jfrog.io/artifactory/api/pypi/python/simple" | |
| pip install "rl-wrapper>=1.0.0" --index-url "https://${{ env.PRODSEC_TOOLS_USER }}:${{ env.PRODSEC_TOOLS_TOKEN }}@a0us.jfrog.io/artifactory/api/pypi/python/simple" |
Contributor
Author
There was a problem hiding this comment.
This is intentional
sanchitmehtagit
approved these changes
Feb 19, 2026
NandanPrabhu
added a commit
that referenced
this pull request
Feb 23, 2026
* main: (79 commits) build(deps): bump aws-actions/configure-aws-credentials from 5.1.1 to 6.0.0 in /.github/actions/rl-scanner (#744) chore: updated the RL wrapper installation path (#746) Release af-v2.0.0-beta.3 (#743) Release afpi-v2.0.0-beta.3 (#742) Release afpi-v2.0.0-beta.3 (#741) build(deps): bump ruby/setup-ruby from 1.286.0 to 1.288.0 in /.github/actions/setup-darwin (#740) CredentialsManager user info/ID token contents accessible via flutter SDK (#607) CI cleanup Avoid running always failing smoke tests for iOS make PR runs with release workflow that triggers flutter analyze udpate pubspec udpate podspecs Release af-v2.0.0-beta.2 pubspec update for beta release Release afpi-v2.0.0-beta.2 fix flutter analyse errors build(deps): bump ruby/setup-ruby from 1.285.0 to 1.286.0 in /.github/actions/setup-darwin (#728) build(deps-dev): bump lodash from 4.17.21 to 4.17.23 in /appium-test (#729) chore:Fixed errors from flutter analyze (#727) feat: Add allowedBrowsers parameter to logout API [SDK-724] (#726) ...
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
📋 Changes
GH action pipeline change for RL scanner