Skip to content

MLE-29694 fixes for vulnerabilities and flaky tests#1080

Open
RitaChen609 wants to merge 4 commits into
developfrom
MLE-29694-review-scans-pipeline
Open

MLE-29694 fixes for vulnerabilities and flaky tests#1080
RitaChen609 wants to merge 4 commits into
developfrom
MLE-29694-review-scans-pipeline

Conversation

@RitaChen609
Copy link
Copy Markdown

@RitaChen609 RitaChen609 commented May 22, 2026

Bumped qs from 6.15.0 to 6.15.2

Increased xdmp lock timeouts in plan-search fragment options tests and made teardown more tolerant

Copilot AI review requested due to automatic review settings May 22, 2026 18:51
@RitaChen609 RitaChen609 changed the title Mle 29694 review scans pipeline MLE-29694 fixes for vulnerabilities and flaky tests May 22, 2026
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR appears aimed at improving CI/pipeline stability and configurability for MarkLogic-related tests and tasks (e.g., reducing test flakiness around lock handling and allowing non-localhost deployments), while also updating a Node dependency patch version.

Changes:

  • Increased MarkLogic lock timeouts in plan-search tests and made lock teardown more tolerant of missing/expired locks.
  • Updated Gradle curl* tasks to use mlHost instead of hardcoding localhost.
  • Bumped qs from 6.15.0 to 6.15.2 (with lockfile update).

Reviewed changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 2 comments.

File Description
test-basic/plan-search.js Adjusts lock acquisition duration and makes teardown lock-release tolerant to errors.
test-app/build.gradle Uses mlHost in curl task URLs instead of localhost.
package.json Updates qs dependency patch version.
package-lock.json Updates lockfile entries for the qs patch version bump.

Comment thread test-basic/plan-search.js Outdated
Comment thread test-basic/plan-search.js Outdated
@RitaChen609 RitaChen609 reopened this May 22, 2026
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.

2 participants