[FEATURE] Add a fixed callback port for browser authentication
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức phù hợp với người mới
- 52/100
- Loại issue
- Tính năng
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Sôi nổi
- Công nghệ
- python
- Lĩnh vực
- authentication, cli, documentation, testing
Hướng nghiên cứu
Bắt đầu bằng cách lần theo parser và lệnh đăng nhập auth đến phần triển khai xác thực cốt lõi. Xem xét việc xử lý lỗi hiện có, tài liệu, các bài kiểm thử và mục Changie được mô tả trong issue. Được xem là hoàn tất khi hành vi và việc xác thực redirect-port hoạt động mà không thay đổi hành vi login mặc định hoặc silent-cache.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Use Case / Problem
Interactive browser authentication currently lets MSAL choose an ephemeral localhost callback port. When Fabric CLI runs on a remote machine over SSH, users cannot reliably preconfigure port forwarding between the local browser and the remote callback listener.
Proposed Solution
Add fab auth login --redirect-port <port> for interactive user authentication.
Expected behavior:
- Accept ports 1 through 65535.
- Skip authentication-method selection and use browser authentication when interaction is required.
- Pass the fixed port to MSAL's local callback listener.
- Temporarily disable Windows/macOS authentication brokers so MSAL honors the port.
- Reject combinations with Azure CLI, managed identity, and service-principal options.
- Return an actionable error when the port cannot be bound.
- Preserve existing silent-cache behavior and all default login behavior when omitted.
Remote usage:
ssh -o ExitOnForwardFailure=yes -L 8400:127.0.0.1:8400 user@remote
BROWSER=echo fab auth login --redirect-port 8400
Alternatives Considered
- Device-code authentication: useful for headless environments but does not provide the same browser callback flow and is tracked separately in #215.
- Relying on an ephemeral port: cannot be forwarded predictably before login.
- Preflight-binding the port: introduces a race; the implementation should let MSAL perform the real bind.
Impact Assessment
- This would help me personally
- This would help my team/organization
- This would help the broader fabric-cli community
Implementation Attestation
- I understand this feature should maintain backward compatibility with existing commands
- I confirm this feature request does not introduce performance regressions for existing workflows
- I acknowledge that new features must follow fabric-cli's established patterns and conventions
Implementation Notes
I have a tested implementation ready in my fork and would like maintainer acknowledgment before opening the PR, per the contribution guide. Please add the help wanted label if the team accepts this contribution. The implementation includes parser, command, core auth, error handling, documentation, tests, and a Changie entry.
- Ngôn ngữ chính
- Python
- Star
- 174
- Fork
- 64
- Merge trung bình
- 6 ngày 38 phút
- Pull request đã merge (30 ngày)
- 5
Chuẩn bị môi trường
Khởi chạy dev container của dự án ngay trên trình duyệt, bằng tài khoản GitHub của bạn.
- 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 microsoft/fabric-cli
-
[FEATURE] Loosen strict dependency pins to improve compatibility with newer `fabric-cicd` versionsCó thể làm lại được Pull request cho issue này đã bị đóng mà không được merge. Đang mởai:feature-triage:unable-to-process enhancement
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 70/100
microsoft/fabric-cli#294 · 1 bình luận · 1 reaction ·
-
[BUG] bulk-export errors with "[NotSupported] Only supported for Lakehouse/Files"Có thể làm lại được @rksharma-owg đã nhận 39 ngày trước và không có pull request nào đang mở. Đang mởai:bug-triage:unable-to-process bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
microsoft/fabric-cli#287 · 2 bình luận ·
-
ai:feature-triage:unable-to-process enhancement
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 64/100
microsoft/fabric-cli#292 · 1 bình luận ·
-
ai:question-triage:unable-to-process question
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 25/100
microsoft/fabric-cli#288 · 2 bình luận ·
-
ai:feature-triage:unable-to-process enhancement
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 42/100
microsoft/fabric-cli#279 · 1 bình luận ·
Tất cả issue của microsoft/fabric-cli
Issue tương tự
-
comp/tools P2 type/bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
NousResearch/hermes-agent#142908 · 1 bình luận ·
-
[oblt-aw][autodoc] Fix malformed headings in new Buildkite docs pagesCó thể đã có người làm @github-actions đã nhận hôm nay. Đang mởdocumentation
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 83/100
Maintainer thường phản hồi trong vòng 1 ngày
-
[Bug]: fileUploadMatrix.test.ts URL case does a real DNS lookup and fails offlineCó thể đã có người làm @r2jaff đã nhận hôm nay. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
ComposioHQ/composio#5054 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 88/100
Maintainer thường phản hồi trong vòng 1 ngày
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 82/100
pytorch/tensordict#3286 ·
Maintainer thường phản hồi trong vòng 1 ngày