Skip to content

Fix Docker API 1.52 compatibility (Docker Engine 29.0.0+)#700

Merged
jesseduffield merged 1 commit intojesseduffield:masterfrom
theasad:fix-docker-api-1.52-compatibility
Nov 14, 2025
Merged

Fix Docker API 1.52 compatibility (Docker Engine 29.0.0+)#700
jesseduffield merged 1 commit intojesseduffield:masterfrom
theasad:fix-docker-api-1.52-compatibility

Conversation

@theasad
Copy link

@theasad theasad commented Nov 12, 2025

Summary

Fixes compatibility issues with Docker Engine 29.0.0+ and Docker API version 1.52.

Changes Made

  • Updated Docker client from v27.1.1 to v28.5.2
  • Updated hardcoded API version from 1.25 (Docker 1.13) to 1.52 (Docker 29.0.0)
  • Added API version negotiation for better forward compatibility
  • Updated minimum Docker version requirement in README

Testing

  • ✅ Successfully builds with new Docker client
  • ✅ Tested with Docker Engine 29.0.0 (API 1.52)
  • ✅ Lazydocker starts and connects to Docker daemon correctly

Fixes

  • Resolves compatibility issues with modern Docker installations
  • Ensures future compatibility through API version negotiation

Breaking Changes

None - backward compatible with older Docker versions through API negotiation.

- Upgrade Docker client from v27.1.1 to v28.5.2
- Update hardcoded API version from 1.25 to 1.52
- Enable API version negotiation for better compatibility
- Fix compatibility with Docker Engine 29.0.0 and newer versions

Resolves issues with lazydocker not working with latest Docker clients.

const (
APIVersion = "1.25"
APIVersion = "1.52"
Copy link

Choose a reason for hiding this comment

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

Unused constant. WithAPIVersionNegotiation() works with older versions too

## Requirements

- Docker >= **1.13** (API >= **1.25**)
- Docker >= **29.0** (API >= **1.52**)
Copy link

Choose a reason for hiding this comment

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

Server: Docker Engine - Community
Engine:
Version: 29.0.0
API version: 1.52 (minimum version 1.24)

@jesseduffield
Copy link
Owner

Thanks for this @theasad . I've made my own additions over at #703

@theasad
Copy link
Author

theasad commented Nov 18, 2025

Hi @jesseduffield, thank you so much for merging my changes! I apologize; I somehow missed the feedback comments from earlier. Thanks again for your time and help.

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