Skip to content

Dap4 Fixes#1543

Merged
lesserwhirls merged 3 commits intoUnidata:maint-5.xfrom
lesserwhirls:dap4
Mar 30, 2026
Merged

Dap4 Fixes#1543
lesserwhirls merged 3 commits intoUnidata:maint-5.xfrom
lesserwhirls:dap4

Conversation

@lesserwhirls
Copy link
Copy Markdown
Member

Description of Changes

Various DAP4 fixes:

  • Accept DMR version 2.0 in addition to 1.0
  • Do not request DAP4 checksums by default
  • Upgrade testcontainers dependency

PR Checklist

  • Link to any issues that the PR addresses
  • Add labels
  • Open as a draft PR
    until ready for review
  • Make sure GitHub tests pass
  • Mark PR as "Ready for Review"

* Both TDS and Hyrax do not return checksums by default, so change DAP4
* checksum default from True to False.  Upgrade Hyrax server test to
  request/not request/not specify checksum responses
@lesserwhirls lesserwhirls added this to the 5.10.0 milestone Mar 30, 2026
@lesserwhirls lesserwhirls added build: gradle Issue or pull request related to the Gradle build infrastructure. internal:dap4 dap4 protocol labels Mar 30, 2026
@lesserwhirls lesserwhirls changed the title Dap4 Dap4 Fixes Mar 30, 2026
@lesserwhirls lesserwhirls marked this pull request as ready for review March 30, 2026 22:47
@lesserwhirls lesserwhirls merged commit 9c5452c into Unidata:maint-5.x Mar 30, 2026
15 checks passed
@lesserwhirls lesserwhirls deleted the dap4 branch March 30, 2026 22:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build: gradle Issue or pull request related to the Gradle build infrastructure. internal:dap4 dap4 protocol

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant