Repository navigation
fix: remediate handlebars, shell-quote, axios, undici, urllib3 and dependency advisories - #757
Open
devin-ai-integration[bot] wants to merge 20 commits into
Open
devin-ai-integration[bot] wants to merge 20 commits into
devin-ai-integration[bot] wants to merge 20 commits into
Conversation
…visories Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Contributor
Author
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
Contributor
☂️ Python Coverage
Overall Coverage
New FilesNo new covered files... Modified FilesNo covered modified files...
|
osm6495
approved these changes
Sep 10, 2026
…manfs/node for security advisories Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
…ities Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
…erabilities Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
…rity-deps Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> # Conflicts: # e2e/react-native/package.json # package.json # yarn.lock
…pc-js and other transitive deps for security advisories Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
This branch has not been deployed
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
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.
Dependency vulnerability fixes (automated)
Generated by the Devin Dependency Security Vuln Fix automation run on 2026-10-12; continues the 2026-09-07, 09-14, 09-21, 09-23, 09-28, 10-01 and 10-05 updates on this branch.
Change type: Minor/patch version bumps — manifest and lockfile changes, no source edits.
Warning
We're not just looking for a review like a normal PR. Security can offer this fix as a recommendation, but doesn't have the tooling or domain knowledge to safely verify changes like this end-to-end for each repo. Please review, test, and own deployment before merging.
Findings addressed
e2e/ruby/rails/demo)The
nextchange updates both root resolutions and the observability-next workspace dev dependency, plus its@next/envand platform-specific SWC lock entries. The Rails locks were regenerated with Bundler 2.5.11 under Ruby 3.3.4 usingbundle lock --update mail --conservative; onlymailmoved. The React Native example has an independent npm lockfile; itsjs-yamlfix is in that lockfile and its manifest overrides. Theadm-zipanddevaluerows were added in this run (rootresolutionsraised to^0.6.1/^5.9.2; Yarn resolved devalue 5.9.4, published 2026-09-18, within the repo's 7-day age gate); other rows carry existing branch fixes. Other already-satisfied alerts on this branch includebaseline-browser-mapping2.11.21, the Vite 6.x descriptor at 6.4.3, and the patched 11.xuuidcopy; older parallel copies remain listed below.Rows from
handlebarsthroughbaseline-browser-mappingwere added in the 2026-10-12 run (rootresolutions/ React Nativeoverridesraised, lockfiles regenerated; urllib3 viapoetry update urllib3 --lockwith Poetry 2.2.1, matching the existing lock headers).Deferred / excluded findings
^1.2.1ranges declared bymetro0.81–0.83 andless(rootyarn.lockande2e/react-native). Needs parent upgrades / owner review.OpenTelemetrycore ≥ 1.16.0, i.e. moving the whole .NET Framework 4.8 OTel stack (1.9–1.11 → 1.16) ine2e/dotnet4/csplus csproj HintPaths; overlaps Dependabot #513, #509, #496. Needs owner-led stack upgrade; can't be built on this Linux machine.rrwebsubmodule. Do not mix breaking migrations into a manifest-only fix.mainby #788 (netty-bom 4.1.137.Final); not duplicated here.buildscriptclasspath, can be pinned the same way in a follow-up.log.KeyValue/log.Valueused bygo/internal/otel/custom_sampler.go,sdk/highlight-go/otel.goand theotelslog/otellogruscontrib bridges; needs source changes and coordinated contrib upgrades by the owning team.^5range declared byget-uri(puppeteer proxy chain).mcp.app.wiz.io) was unreachable from this run, so the 2026-10-12 update is based on Dependabot only.No new PR group was deferred solely by the three-new-PR cap; this PR is an update to an existing security dependency PR.
Verification
Re-run on 2026-10-12 after merging
main(conflicts inpackage.json,e2e/react-native/package.json,yarn.lockresolved keeping the higher of both sides' pins):PUPPETEER_SKIP_DOWNLOAD=1 YARN_NPM_REGISTRY_SERVER=https://registry.npmjs.org node .yarn/releases/yarn-4.13.0.cjs install --immutable --mode=skip-build(default 7-day age gate); ✅yarn dedupe --check.yarn build:sdk(22/22).yarn turbo run test --filter './sdk/*' --filter './sdk/@launchdarkly/*'(52/52 tasks).yarn lint(50/50); ✅prettier --checkon changed manifests/lockfiles (Huskypretty-quick --stagedalso ran on each commit).handlebars4.7.10 was published 2026-10-05 (6.6 days before this run); it is a Critical fix, so the lock was resolved with a 6-day local age gate. Other new lock entries are ≥7 days old.Re-run on 2026-10-05 after merging
main(clean merge) and adding the piscina/proxy-addr/rubyzip bumps:piscinaandproxy-addrare fixed via rootresolutions("piscina": "^4.9.4","proxy-addr": "^2.0.8"); the lockfile diff is limited to those two entries (proxy-addr is a transitive ofexpress4.22 / 5.x).rubyzipine2e/ruby/rails/demo/Gemfile.lockwas moved to 3.4.0 (no runtime deps; same version already indemo-rails70). Ruby 3.3.4 isn't available here (Docker Hub blob CDN blocked), so the edit was cross-checked with a Bundler 2.4bundle lockre-resolve, which kept rubyzip at 3.4.0 and changed nothing else apart from Bundler-version platform naming. CI's Rails E2E, Rails Example App, and Dockerized Ruby jobs all pass on this head.PUPPETEER_SKIP_DOWNLOAD=1 YARN_NPM_REGISTRY_SERVER=https://registry.npmjs.org node .yarn/releases/yarn-4.13.0.cjs install --immutable --mode=skip-build(no age-gate override needed).yarn build:sdk(20/20).yarn turbo run test --filter './sdk/*' --filter './sdk/@launchdarkly/*'(50/50 tasks).yarn lint(49/49); ✅prettier --check package.json(equivalent of the Huskypretty-quick --stagedhook).2026-10-01: merged
main(includes #788). The Netty pin briefly added here was removed because it duplicated #788; this PR no longer touches anybuild.gradlefile.Re-run on 2026-09-28 after merging
main(clean merge) and adding the adm-zip/devalue bump:PUPPETEER_SKIP_DOWNLOAD=1 YARN_NPM_REGISTRY_SERVER=https://registry.npmjs.org node .yarn/releases/yarn-4.13.0.cjs install --immutable --mode=skip-build(no age-gate override needed this run).yarn build:sdk(20/20 SDK packages).yarn turbo run test --filter '@launchdarkly/*' --filter highlight.run(27/27 tasks).yarn lint(49/49 tasks); ✅prettier --check package.json(equivalent of the Huskypretty-quick --stagedhook).bundle exec rails test(API-only 3 tests/5 assertions), React Native 111 tests,@highlight-run/next15/15; rootyarn teststill stops on Angular's blockedfonts.googleapis.com.Previous CI on head
2be2bd1e7: 36/37 checks pass. The only red check,Require additional human oversight on bot PRs, is an expectedaction_requiredhuman approval gate, not a code failure. Reviewers:team-observability(CODEOWNERS) andteam-product-security.Link to Devin session: https://app.devin.ai/sessions/4a98d40b0c594f6ebb281f63d1be41ea
Open in Devin Desktop: https://app.devin.ai/desktop/session/4a98d40b0c594f6ebb281f63d1be41ea?variant=devin