Support `psql` as part of the server host
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- Mức phù hợp với người mới
- 45/100
- Loại issue
- Lỗi
- Độ rõ ràng
- Đặc tả rõ ràng
- Mức độ hoạt động
- Đình trệ
- Công nghệ
- github-actions, postgresql, typescript
Hướng nghiên cứu
Bắt đầu trong src/Utils/ActionInputs.ts, tại logic được liên kết để loại bỏ psql và dấu ngoặc kép khỏi chuỗi kết nối. Tái hiện sự cố với một host như caf-named-db-psql.postgres.database.azure.com, sau đó xác minh rằng action giữ lại psql khi nó là một phần của host, đồng thời vẫn phân tích cú pháp chuỗi kết nối chính xác.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
According to the CAF (Cloud Adoption Framework) the official abbreviation for a PostgreSQL database is psql.
But the current implementation blindly replaces 'psql' with '', which breaks the ability to use this action with such a named DB.
Input:
- uses: azure/postgresql@v1
with:
connection-string: host=caf-named-db-psql.postgres.database.azure.com ...
Output:
psql: error: could not translate host name "caf-named-db-.postgres.database.azure.com" to address: Name or service not known
Error: Error occurred while running action:
Error: The process '/usr/bin/psql' failed with exit code 2
Personal workaround:
The code currently not only replaces 'psql' with '', it also (as a second step) replaces '"' with ''. So a functional workaround that I use is adding a rouge " like this:
- uses: azure/postgresql@v1
with:
connection-string: host=caf-named-db-p"sql.postgres.database.azure.com ...
But this is far from ideal 😂
- Ngôn ngữ chính
- TypeScript
- Star
- 25
- Fork
- 22
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Hướng dẫn đóng góp
Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này
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/postgresql
-
Upgrade to Node.js 24 Đang mởneed-to-triage
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
Azure/postgresql#58 ·
-
need-to-triage
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 35/100
Azure/postgresql#55 ·
-
need-to-triage
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 20/100
Azure/postgresql#53 · 1 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 25/100
Azure/postgresql#52 · 1 bình luận ·
-
need-to-triage
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 15/100
Azure/postgresql#41 ·
Tất cả issue của Azure/postgresql
Issue tương tự
-
VerificationGate: ATTRIBUTION quote guard never matches a normal quotation (\b around the quote) Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
danielmiessler/LifeOS#2234 ·
-
T: Bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 85/100
-
Mend: dependency security vulnerability untriaged
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100