iris-devtester 1.12.0__tar.gz → 1.12.2__tar.gz

{iris_devtester-1.12.0 → iris_devtester-1.12.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: iris-devtester
-Version: 1.12.0
+Version: 1.12.2
 Summary: Battle-tested InterSystems IRIS infrastructure utilities for Python development
 Author-email: InterSystems Community <community@intersystems.com>
 Maintainer-email: Thomas Dyar <thomas.dyar@intersystems.com>

iris_devtester-1.12.2/docs/learnings/iris-connection-access-denied-bug.md

@@ -0,0 +1,37 @@
+# Learning: IRIS Connection "Access Denied" / "Invalid Message received"
+
+## Symptoms
+When connecting to an InterSystems IRIS container using `iris.connect` (Python Native SDK / DB-API), the connection fails with:
+```
+<COMMUNICATION LINK ERROR> Failed to connect to server;
+Details: <COMMUNICATION ERROR> Invalid Message received;
+Details: Access Denied
+```
+This happens even if the port, username, and password appear correct, and direct connections to `localhost` work.
+
+## Root Cause
+The "Invalid Message received" error is a specific response from the IRIS Superserver when it receives data that does not match the expected wire protocol (e.g., hitting the Superserver port with an HTTP request, or hitting the Web port with a Superserver client).
+
+In `iris-devtester`, `IRISContainer.get_config()` was using `get_container_host_ip()` to determine the host. On some platforms or Docker configurations, this can return `0.0.0.0`. While `0.0.0.0` is valid for *listening*, it is often problematic for *connecting* on certain OSs or when IRIS security policies restrict connections to specific interfaces. More importantly, if the discovery logic returned the **Web Port** (52773) instead of the **Superserver Port** (1972) due to misconfiguration or "attached" container inspection errors, the Superserver would reject the "invalid" protocol message.
+
+## Solution
+Sanitize the discovered host and ensure the Superserver port is correctly prioritized over other mapped ports.
+
+In `iris_devtester/containers/iris_container.py`:
+```python
+def get_config(self) -> IRISConfig:
+    ...
+    discovered_host = self.get_container_host_ip()
+    # Sanitize 0.0.0.0 to localhost for client connections
+    if discovered_host in ("0.0.0.0", "::"):
+        discovered_host = "localhost"
+    
+    self.host = discovered_host
+    self._mapped_port = int(self.get_exposed_port(1972))
+    ...
+```
+
+## Prevention
+1. **Explicit Port Discovery**: Always use the internal port (1972) as the key for `get_exposed_port()` discovery, rather than assuming the first available port.
+2. **Host Sanitization**: Always translate `0.0.0.0` or `::` to `localhost` for client-side connection strings.
+3. **Validation**: Validate that `container_name` is never `None` during `attach()` calls to ensure Docker SDK inspection succeeds.

{iris_devtester-1.12.0 → iris_devtester-1.12.2}/iris_devtester/__init__.py

@@ -26,7 +26,7 @@ LangChain Integration:
     ...     # Build your RAG app...
 """
 
-__version__ = "1.10.3"
+__version__ = "1.12.2"
 __author__ = "InterSystems Community"
 __license__ = "MIT"
 

{iris_devtester-1.12.0 → iris_devtester-1.12.2}/iris_devtester/cli/container.py

@@ -80,8 +80,11 @@ def container_group(ctx):
     "--timeout", type=int, default=60, help="Health check timeout in seconds (default: 60)"
 )
 @click.option("--cpf", help="Path to CPF merge file or raw CPF content")
+@click.option(
+    "--auto-port", is_flag=True, help="Automatically find and assign free ports via registry"
+)
 @click.pass_context
-def up(ctx, config, name, edition, image, license_key, detach, timeout, cpf):
+def up(ctx, config, name, edition, image, license_key, detach, timeout, cpf, auto_port):
     """
     Create and start IRIS container from configuration.
 
@@ -161,6 +164,37 @@ def up(ctx, config, name, edition, image, license_key, detach, timeout, cpf):
             container_config.cpf_merge = cpf
             click.echo(f"  → CPF Merge: {cpf[:50]}...")
 
+        # Handle auto-port assignment (Feature 027 - Smart Port Fallback)
+        if auto_port:
+            from iris_devtester.ports.registry import PortRegistry
+
+            registry = PortRegistry()
+            project_path = str(Path.cwd().absolute())
+
+            click.echo(f"⏳ Engaging port registry for project: {project_path}")
+            # Use current config port as preferred
+            preferred = container_config.superserver_port
+            assignment = registry.assign_port(project_path, preferred_port=preferred)
+
+            if assignment.port != preferred:
+                click.echo(
+                    click.style(
+                        f"⚠️  Port {preferred} was unavailable. Assigned {assignment.port} instead.",
+                        fg="yellow",
+                    )
+                )
+            else:
+                click.echo(f"✓ Port {assignment.port} assigned and verified.")
+
+            container_config.superserver_port = assignment.port
+            # Update web port proportionally if it's the default
+            if container_config.webserver_port == 52773:
+                # Simple logic: if superserver shifted by N, shift webserver by N
+                offset = assignment.port - 1972
+                if offset > 0:
+                    container_config.webserver_port = 52773 + offset
+                    click.echo(f"  → Web port adjusted to {container_config.webserver_port}")
+
         # Check if container already exists
         existing_container = IRISContainerManager.get_existing(container_config.container_name)
 
@@ -475,8 +509,11 @@ def list_containers(ctx, show_all, output_format):
 @click.option(
     "--timeout", type=int, default=60, help="Health check timeout in seconds (default: 60)"
 )
+@click.option(
+    "--auto-port", is_flag=True, help="Automatically find and assign free ports via registry"
+)
 @click.pass_context
-def start(ctx, container_name, config, timeout):
+def start(ctx, container_name, config, timeout, auto_port):
     """
     Start existing IRIS container or create new one.
 
@@ -517,8 +554,40 @@ def start(ctx, container_name, config, timeout):
                 else:
                     container_config = ContainerConfig.default()
 
-            # Create and start container using Docker SDK (Feature 011 - T015)
-            click.echo("⏳ Configuring and starting container with Docker SDK...")
+                # Override name
+                container_config.container_name = container_name
+
+                # Handle auto-port assignment (Feature 027 - Smart Port Fallback)
+                if auto_port:
+                    from iris_devtester.ports.registry import PortRegistry
+
+                    registry = PortRegistry()
+                    project_path = str(Path.cwd().absolute())
+
+                    click.echo(f"⏳ Engaging port registry for project: {project_path}")
+                    preferred = container_config.superserver_port
+                    assignment = registry.assign_port(project_path, preferred_port=preferred)
+
+                    if assignment.port != preferred:
+                        click.echo(
+                            click.style(
+                                f"⚠️  Port {preferred} was unavailable. Assigned {assignment.port} instead.",
+                                fg="yellow",
+                            )
+                        )
+                    else:
+                        click.echo(f"✓ Port {assignment.port} assigned and verified.")
+
+                    container_config.superserver_port = assignment.port
+                    # Update web port proportionally if it's the default
+                    if container_config.webserver_port == 52773:
+                        offset = assignment.port - 1972
+                        if offset > 0:
+                            container_config.webserver_port = 52773 + offset
+                            click.echo(f"  → Web port adjusted to {container_config.webserver_port}")
+
+                # Create and start container using Docker SDK (Feature 011 - T015)
+                click.echo("⏳ Configuring and starting container with Docker SDK...")
 
             # Validate volume paths before creation
             volume_errors = container_config.validate_volume_paths()

{iris_devtester-1.12.0 → iris_devtester-1.12.2}/iris_devtester/config/discovery.py

@@ -9,6 +9,7 @@ Automatically discovers IRIS configuration from multiple sources:
 5. Sensible defaults (lowest priority)
 """
 
+import logging
 import os
 from pathlib import Path
 from typing import Any, Dict, Optional, Union
@@ -25,6 +26,8 @@ from iris_devtester.config.defaults import (
 )
 from iris_devtester.config.models import IRISConfig
 
+logger = logging.getLogger(__name__)
+
 
 def discover_config(explicit_config: Optional[IRISConfig] = None) -> IRISConfig:
     """

{iris_devtester-1.12.0 → iris_devtester-1.12.2}/iris_devtester/containers/iris_container.py

@@ -126,7 +126,8 @@ class IRISContainer(IRISBase):
         project_ns = get_project_namespace()
         kwargs.setdefault("namespace", project_ns)
         
-        return cls.attach(instance.name, **kwargs)
+        container_name = str(instance.name) if instance.name else "iris_db"
+        return cls.attach(container_name, **kwargs)
 
     @classmethod
     def community(
@@ -245,6 +246,9 @@ class IRISContainer(IRISBase):
         Returns:
             An IRISContainer instance attached to the existing container.
         """
+        if not container_name:
+            raise ValueError("container_name must be a non-empty string")
+
         instance = cls(image="", name=container_name, **kwargs)
         instance._is_attached = True
         instance._container_name = container_name
@@ -268,10 +272,18 @@ class IRISContainer(IRISBase):
                 instance.get_config()
 
         except Exception as e:
-            logger.warning(f"Could not fully inspect container '{container_name}': {e}")
-            # Fallback to defaults
-            instance.host = instance.host or "localhost"
-            instance._mapped_port = instance._mapped_port or 1972
+            # For attached containers, we MUST find the container to be useful
+            raise ValueError(
+                f"Container '{container_name}' not found or not running\n"
+                "\n"
+                "What went wrong:\n"
+                f"  {str(e)}\n"
+                "\n"
+                "How to fix it:\n"
+                "  1. Verify the container name is correct: docker ps\n"
+                "  2. Ensure the container is started: docker start <name>\n"
+                "  3. Check if your user has permission to access the Docker socket"
+            )
 
         return instance
 
@@ -455,21 +467,29 @@ class IRISContainer(IRISBase):
         """
         # Always create fresh config to pick up any credential changes
         # (e.g., conftest may update _username/_password after start())
-        self._config = IRISConfig(
+        config = IRISConfig(
             username=self._username,
             password=self._password,
             namespace=self._namespace,
             container_name=self.get_container_name(),
         )
-        config = self._config
+        self._config = config
         try:
-            # Get host and mapped port from testcontainers
+            # Get host and mapped port
             # IMPORTANT: self.port must remain 1972 (internal port) for get_exposed_port() to work
-            self.host = self.get_container_host_ip()
+            
+            # Use 'localhost' as default host for local containers
+            discovered_host = self.get_container_host_ip()
+            if discovered_host in ("0.0.0.0", "::"):
+                discovered_host = "localhost"
+            
+            self.host = discovered_host
             self._mapped_port = int(self.get_exposed_port(1972))  # Use internal port to get mapping
+            
             config.host = self.host
             config.port = self._mapped_port  # Config uses the host-mapped port for connections
         except Exception:
+            # If discovery fails, stick with defaults (localhost:1972)
             pass
         return config
 
@@ -527,6 +547,36 @@ class IRISContainer(IRISBase):
         self._password = password
         return self
 
+    def reset_password(
+        self, username: str = "_SYSTEM", new_password: str = "SYS", timeout: int = 30
+    ) -> bool:
+        """
+        Reset password for a user in this container.
+
+        Args:
+            username: Username to reset
+            new_password: New password to set
+            timeout: Timeout in seconds
+
+        Returns:
+            True if successful
+        """
+        from iris_devtester.utils.password import reset_password as reset_func
+
+        config = self.get_config()
+        result = reset_func(
+            container_name=self.get_container_name(),
+            username=username,
+            new_password=new_password,
+            timeout=timeout,
+            hostname=config.host,
+            port=config.port,
+        )
+        if result.success:
+            self._password = new_password
+            return True
+        return False
+
     def start(self) -> "IRISContainer":
         """Start container with pre-config support and port registry integration."""
         if self._preconfigure_password:

{iris_devtester-1.12.0 → iris_devtester-1.12.2}/iris_devtester.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: iris-devtester
-Version: 1.12.0
+Version: 1.12.2
 Summary: Battle-tested InterSystems IRIS infrastructure utilities for Python development
 Author-email: InterSystems Community <community@intersystems.com>
 Maintainer-email: Thomas Dyar <thomas.dyar@intersystems.com>

{iris_devtester-1.12.0 → iris_devtester-1.12.2}/iris_devtester.egg-info/SOURCES.txt

@@ -56,6 +56,7 @@ docs/learnings/embedded-python-considerations.md
 docs/learnings/feature-002-pivot-summary.md
 docs/learnings/integration-test-dbapi-limitation.md
 docs/learnings/iris-backup-patterns.md
+docs/learnings/iris-connection-access-denied-bug.md
 docs/learnings/iris-container-performance.md
 docs/learnings/iris-container-readiness.md
 docs/learnings/iris-docker-sdk-environment-variables.md

{iris_devtester-1.12.0 → iris_devtester-1.12.2}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "iris-devtester"
-version = "1.12.0"
+version = "1.12.2"
 description = "Battle-tested InterSystems IRIS infrastructure utilities for Python development"
 readme = "README.md"
 requires-python = ">=3.9"