fix: Platform--instance packing problme

Signed-off-by: Khushiyant <khushiyant2002@gmail.com>

Author: Khushiyant

docker/api/image.py

@@ -495,7 +495,10 @@ def push(self, repository, tag=None, stream=False, auth_config=None,
                 raise errors.InvalidVersion(
                     'platform was only introduced in API version 1.46'
                 )
-            params['platform'] = Platform
+            # Handle both Platform instances and dict inputs
+            if isinstance(platform, dict):
+                platform = Platform(**platform)
+            params['platform'] = platform
 
         response = self._post_json(
             u, None, headers=headers, stream=stream, params=params

docker/types/__init__.py

@@ -1,6 +1,7 @@
 from .containers import ContainerConfig, DeviceRequest, HostConfig, LogConfig, Ulimit
 from .daemon import CancellableStream
 from .healthcheck import Healthcheck
+from .image import Platform
 from .networks import EndpointConfig, IPAMConfig, IPAMPool, NetworkingConfig
 from .services import (
     ConfigReference,

docker/types/image.py

@@ -3,33 +3,33 @@
 
 class Platform(DictType):
     def __init__(self, **kwargs):
-        architecture = kwargs.get('architecture', kwargs.get('Architecture'))
-        os = kwargs.get('os', kwargs.get('OS'))
+        architecture = kwargs.get('architecture')
+        os = kwargs.get('os')
 
         if architecture is None and os is None:
             raise ValueError("At least one of 'architecture' or 'os' must be provided")
 
 
         super().__init__({
-            'Architecture': architecture,
-            'OS': os,
-            'OSVersion': kwargs.get('os_version', kwargs.get('OSVersion')),
-            'OSFeatures': kwargs.get('os_features', kwargs.get('OSFeatures')),
-            'Variant': kwargs.get('variant', kwargs.get('Variant'))
+            'architecture': architecture,
+            'os': os,
+            'os_version': kwargs.get('os_version'),
+            'os_features': kwargs.get('os_features'),
+            'variant': kwargs.get('variant')
         })
 
     @property
     def architecture(self):
-        return self['Architecture']
+        return self['architecture']
 
     @property
     def os(self):
-        return self['OS']
+        return self['os']
 
     @architecture.setter
     def architecture(self, value):
-        self['Architecture'] = value
+        self['architecture'] = value
 
     @os.setter
     def os(self, value):
-        self['OS'] = value
+        self['os'] = value

tests/unit/fake_api.py

@@ -1,4 +1,5 @@
 from docker import constants
+from docker.types.image import Platform
 
 from . import fake_stat
 
@@ -21,7 +22,8 @@
 FAKE_SECRET_NAME = 'super_secret'
 FAKE_CONFIG_ID = 'sekvs771242jfdjnvfuds8232'
 FAKE_CONFIG_NAME = 'super_config'
-FAKE_PLATFORM = "{'os': 'linux','architecture': 'arm','variant': 'v5'}"
+
+FAKE_PLATFORM = Platform(os='linux', architecture='arm', variant='v5')
 
 # Each method is prefixed with HTTP method (get, post...)
 # for clarity and readability