Skip to content

Commit a90e2fa

Browse files
FIX: Address Copilot review - fix exception hierarchy, Linux archs, security notes
1 parent 4e2515b commit a90e2fa

2 files changed

Lines changed: 16 additions & 9 deletions

File tree

.github/copilot-instructions.md

Lines changed: 13 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -30,6 +30,8 @@ This repository includes detailed prompt files for common tasks. Reference these
3030

3131
## Usage Examples (For Suggesting to Users)
3232

33+
> **Security Note**: Examples use `TrustServerCertificate=yes` for local development with self-signed certificates. For production, remove this option to ensure proper TLS certificate validation.
34+
3335
### Basic Connection and Query
3436
```python
3537
import mssql_python
@@ -259,14 +261,17 @@ The `ddbc_bindings.py` module implements sophisticated architecture detection:
259261
Critical for error handling guidance:
260262

261263
```
262-
Error (base)
263-
├── DatabaseError
264-
│ ├── InterfaceError # Driver/interface issues
265-
│ ├── OperationalError # Connection/timeout issues
266-
│ ├── IntegrityError # Constraint violations
267-
│ ├── ProgrammingError # SQL syntax errors
268-
│ └── DataError # Invalid data processing
269-
└── Warning
264+
Exception (base)
265+
├── Warning
266+
└── Error
267+
├── InterfaceError # Driver/interface issues
268+
└── DatabaseError
269+
├── DataError # Invalid data processing
270+
├── OperationalError # Connection/timeout issues
271+
├── IntegrityError # Constraint violations
272+
├── InternalError # Internal driver/database errors
273+
├── ProgrammingError # SQL syntax errors
274+
└── NotSupportedError # Unsupported features/operations
270275
```
271276

272277
## Critical Anti-Patterns (DO NOT)

llms.txt

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,6 +52,8 @@ sudo ACCEPT_EULA=Y zypper install -y msodbcsql18
5252

5353
## Quick Start
5454

55+
> **Security Note**: The examples below use `TrustServerCertificate=yes` for local development with self-signed certificates. For production or remote connections, remove this option to ensure proper TLS certificate validation.
56+
5557
```python
5658
import mssql_python
5759

@@ -295,7 +297,7 @@ conn = mssql_python.connect("SERVER=...;DATABASE=...")
295297

296298
- Windows (x64, ARM64)
297299
- macOS (ARM64, x86_64)
298-
- Linux: Debian, Ubuntu, RHEL, SUSE (x64), Alpine (musl)
300+
- Linux (x86_64, ARM64): Debian, Ubuntu, RHEL, SUSE, Alpine (musl)
299301

300302
## Supported Python Versions
301303

0 commit comments

Comments
 (0)