Database names with spaces cause firewall detection to fail - sqlcmd argument not quoted
Đánh giá
Issue này chưa được đánh giá.
Mô tả
When the connection string contains a database name with a space (e.g. Initial Catalog=MyDatabase Live), the firewall IP detection step fails because the database name is not properly quoted when passed to sqlcmd.
Error:
Failed to add firewall rule. Unable to detect client IP Address. Sqlcmd: 'Live': Unknown command. Enter '--help' for command help.
Debug output:
##[debug]Validating if client has access to 'MyDatabase Live' on 'myserver.database.windows.net'.
##[debug]The process 'sqlcmd' failed with exit code 1
##[debug]SqlCmd stderr: Sqlcmd: 'Live': Unknown command. Enter '--help' for command help.
The action correctly parses the database name as MyDatabase Live, but fails to quote it when executing sqlcmd.
Environment:
azure/[email protected]runs-on: ubuntu-latest- Azure SQL Database with
Authentication=Active Directory Default
Connection string:
Server=myserver.database.windows.net;Initial Catalog=MyDatabase Live;Authentication=Active Directory Default;Encrypt=True;TrustServerCertificate=False;Connection Timeout=30;
Attempted workarounds (none worked):
- Single quotes:
Initial Catalog='MyDatabase Live' - Double quotes:
Initial Catalog="MyDatabase Live"
Potential root cause:
The detectIPAddress function extracts the database name correctly but may pass it to sqlcmd as the -d argument without quoting, resulting in:
sqlcmd -S myserver.database.windows.net -d MyDatabase Live -Q "select getdate()"
Expected behaviour:
sqlcmd -S myserver.database.windows.net -d "MyDatabase Live" -Q "select getdate()"
- Ngôn ngữ chính
- TypeScript
- Star
- 142
- Fork
- 73
- Merge trung bình
- 3 ngày 3 phút
- Pull request đã merge (30 ngày)
- 1
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- Có mẫu pull request
- Đọc hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của Azure/sql-action
-
Bump Major Tag VersionĐang mởneed-to-triage
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 62/100
Azure/sql-action#247 ·
-
need-to-triage
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 65/100
Azure/sql-action#282 ·
-
need-to-triage
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 45/100
Azure/sql-action#262 · 2 bình luận ·
-
need-to-triage
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 35/100
Azure/sql-action#261 ·
-
need-to-triage
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 48/100
Azure/sql-action#256 · 1 bình luận ·
Tất cả issue của Azure/sql-action
Issue tương tự
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
AOSSIE-Org/DebateAI#611 ·
Maintainer thường phản hồi trong vòng 3 ngày
-
Upgrade node-libzim to 4.7.0Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
openzim/mwoffliner#2933 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Use the README category name for website links and submissionsCó thể đã có người làm @dajiaohuang đã nhận hôm nay. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
birobirobiro/awesome-shadcn-ui#647 ·
Maintainer thường phản hồi trong vòng 2 ngày
-
Twake Drive picker: closePicker() never destroys the intent (stop() is on the promise returned by start(), not by create())Có thể đã có người làm @chibenwa đã nhận hôm nay. Đang mởclaude
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 84/100
linagora/twake-calendar-frontend#1498 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Add: Valea Prahovei TV RO SDĐang mởcheck:passed streams:add
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
Maintainer thường phản hồi trong vòng 1 ngày