Skip to content

DX-20927 | Back-merge master into development (reconcile rewritten history) - #314

Open
reeshika-h wants to merge 319 commits into
developmentfrom
back-merge/DX-20927
Open

reeshika-h wants to merge 319 commits into
developmentfrom
back-merge/DX-20927

Conversation

@reeshika-h

Copy link
Copy Markdown
Contributor

Why

The secret-removal history rewrite on master changed its commit SHAs, while development stayed on the old pre-rewrite commits. The two branches' last shared commit is now from Oct 2024, and GitHub shows master 318 ahead / 307 behind.

What this does

I checked every development-only commit: each is a content-identical copy of a master commit (same tree and same message; only the GPG signature was dropped by the rewrite). development has no unique changes.

This branch is master + a merge of development using the ours strategy:

  • The resulting tree is exactly master's tree.
  • development becomes a parent again, so the branches share a merge-base and future back-merges/releases are normal.
  • Against development, the diff is only master's new changes (jsoup fixes, version/changelog, CODEOWNERS, workflows).

Note: after the next release, the ~307 old duplicate commits will show up in master's history. They contain no secrets (a gitleaks scan of the development-only range found nothing).

Jira: DX-20927

🤖 Generated with Claude Code

Aravind-Kumar-cstk and others added 30 commits April 23, 2025 21:37
- Introduced SanityReport class to generate test summaries and send reports to Slack.
- Updated logging in CSHttpConnection to use logger instead of printStackTrace.
- Modified error handling in Entry class to check for empty error messages.
- Updated dependency versions in pom.xml and added new dependencies.
- Added test configuration properties to .gitignore.
- Created send-report.sh script for running tests and sending reports.
Fixed testcase assertions to be env agnostic
Update version to 2.1.1 and updated changelog
Update branch restrictions in PR workflow and add Talisman configuration
reeshika-h and others added 25 commits June 24, 2026 15:01
chore: back-merge master into development
fix: update version to 2.7.2 and add Snyk fixes to CHANGELOG
…anch headers

Addresses Copilot review on PR #295:
- Guard variants(String) and variants(String[]) against null input
- Treat whitespace-only variant/branch as blank via trim().isEmpty()
- Join multi-variant header with "," for consistency with other SDK headers

Blank input intentionally skips (does not remove) the header: headers are a
shared Stack-level map, so remove() would clear the header for sibling
Entry/Query objects and the Stack-level default.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
…-support

Add variant and branch support in Entry and Query APIs
…odel

Snyk Code flagged publish_details "user" test-fixture fields as
hardcoded credentials (CWE-798). These are non-secret string values
used only to populate mock JSON payloads, so add deepcode ignore
annotations instead of leaving unscoped comments.
jsoup 1.22.x is pulled in transitively via com.contentstack.sdk:utils
and is flagged by Snyk for a Cross-site Scripting (XSS) issue, fixed
in 1.23.1. Pin the version in dependencyManagement following the
existing pattern used for other contentstack-utils transitive deps.
fix: suppress Snyk hardcoded-credential false positives in tests
fix: pin jsoup to 1.23.1 to resolve XSS vulnerability
…nerability

jsoup is pulled in transitively via com.contentstack.sdk:utils and is
flagged by Snyk (CVE-2026-75140) for unbounded memory growth in
XmlTreeBuilder's namespace scope tracking. Fixed upstream in 1.23.2 via
a rewritten NamespaceBindings scope tracker. Bump the dependencyManagement
pin following the existing pattern used for other contentstack-utils
transitive deps.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
fix: bump jsoup pin to 1.23.2 to resolve Snyk resource allocation vulnerability
The secret-removal history rewrite on master left development on the old,
pre-rewrite commit SHAs. Every development-only commit is a content-identical
copy of a master commit (same tree and message, only the GPG signature differs),
so this merge records development as a parent while keeping master's tree
unchanged. That gives the two branches a shared merge-base again.

Refs DX-20927

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@reeshika-h
reeshika-h requested a review from a team as a code owner September 24, 2026 09:43
@snyk-io

snyk-io Bot commented Sep 24, 2026 •

Copy link
Copy Markdown

✅ Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
✅ Open Source Security 0 0 0 0 0 issues
✅ Licenses 0 0 0 0 0 issues
✅ Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@reeshika-h

Copy link
Copy Markdown
Contributor Author

⚠️ Please merge with "Create a merge commit", not squash or rebase. Squash or rebase would drop the development parent, the branches would still have no shared merge-base, and the divergence would stay.

@github-actions

Copy link
Copy Markdown

🔒 Security Scan Results

ℹ️ Note: Only vulnerabilities with available fixes (upgrades or patches) are counted toward thresholds.

Check Type Count (with fixes) Without fixes Threshold Result
🔴 Critical Severity 0 0 10 ✅ Passed
🟠 High Severity 0 4 25 ✅ Passed
🟡 Medium Severity 0 1 500 ✅ Passed
🔵 Low Severity 0 0 1000 ✅ Passed

⏱️ SLA Breach Summary

✅ No SLA breaches detected. All vulnerabilities are within acceptable time thresholds.

Severity Breaches (with fixes) Breaches (no fixes) SLA Threshold (with/no fixes) Status
🔴 Critical 0 0 15 / 30 days ✅ Passed
🟠 High 0 0 30 / 120 days ✅ Passed
🟡 Medium 0 0 90 / 365 days ✅ Passed
🔵 Low 0 0 180 / 365 days ✅ Passed

ℹ️ Vulnerabilities Without Available Fixes (Informational Only)

The following vulnerabilities were detected but do not have fixes available (no upgrade or patch). These are excluded from failure thresholds:

  • Critical without fixes: 0
  • High without fixes: 4
  • Medium without fixes: 1
  • Low without fixes: 0

✅ BUILD PASSED - All security checks passed

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.

8 participants