Skip to content

MLE-31424: Backport to 11.3 changing xdmp:is-forwarded to xdmp:request-forwarded - #612

Merged
vzhang-progress merged 1 commit into
develop-11.3from
MLE-31424-backport-name-change
Jul 24, 2026
Merged

MLE-31424: Backport to 11.3 changing xdmp:is-forwarded to xdmp:request-forwarded#612
vzhang-progress merged 1 commit into
develop-11.3from
MLE-31424-backport-name-change

Conversation

@vzhang-progress

Copy link
Copy Markdown
Contributor
  • Change xdmp:is-forwarded to xdmp:request-forwarded as the builtin name changed in xdmp
  • Tests: Unit tests all passed

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Updates mlcp’s reverse-proxy detection logic to use the renamed MarkLogic builtin xdmp:request-forwarded (replacing xdmp:is-forwarded) within the XQuery snippets embedded in ContentOutputFormat, aligning with MarkLogic 11.3’s builtin naming.

Changes:

  • Switched the HTTP and XDBC forwarded-request detection queries from xdmp:is-forwarded to xdmp:request-forwarded.
  • Kept the existing header-based fallback behavior when the function lookup fails.
Comments suppressed due to low confidence (1)

src/main/java/com/marklogic/mapreduce/ContentOutputFormat.java:138

  • Same as HEADER_QUERY: XDBC_HEADER_QUERY no longer attempts xdmp:is-forwarded() when present. If xdmp:request-forwarded is unavailable on the target server, this changes behavior compared to earlier releases. Consider looking up both names before falling back to request-header checks.
        "let $f := fn:function-lookup(xs:QName('xdmp:request-forwarded'),0)\n" +
        "return if (exists($f)) then $f() " +
        "else (" +

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/main/java/com/marklogic/mapreduce/ContentOutputFormat.java
@vzhang-progress
vzhang-progress merged commit 2caaf45 into develop-11.3 Jul 24, 2026
4 checks passed
@vzhang-progress
vzhang-progress deleted the MLE-31424-backport-name-change branch July 24, 2026 23:57
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.

3 participants