Skip to content

fix: --output_format respects argument #56

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
Alon Yeshurun (ayeshurun) merged 11 commits into from
Nov 17, 2025
Merged

fix: --output_format respects argument #56

Show file tree
Hide file tree
Changes from 10 commits
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
fdde498
fix: --output_format respects argument
Guust-Franssens Nov 1, 2025
2f54a14
Keep text output spacing exactly as before
Guust-Franssens Nov 1, 2025
1eb7552
chore: add changie
Guust-Franssens Nov 3, 2025
2bf97fe
chore: changie should be present tense
Guust-Franssens Nov 3, 2025
7250551
chore: refactor to let fab_ui handle the output format
Guust-Franssens Nov 13, 2025
e042007
flatten dictionary for better fab_ui printing
Guust-Franssens Nov 13, 2025
a69c299
remove unnecessary comment
Guust-Franssens Nov 13, 2025
7c086c5
add-in login status stderr message
Guust-Franssens Nov 16, 2025
8752a8d
remove unused import
Guust-Franssens Nov 16, 2025
98d8e3d
change user_id -> principal_id
Guust-Franssens Nov 17, 2025
193b6ac
fix: update test_auth_status for new output format
Guust-Franssens Nov 17, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions .changes/unreleased/fixed-20251103-083447.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
kind: fixed
body: Fix `--output_format` argument for command `fab auth status`
time: 2025-11-03T08:34:47.3979808+01:00
28 changes: 16 additions & 12 deletions src/fabric_cli/commands/auth/fab_auth.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
from argparse import Namespace
from typing import Any, Optional

from fabric_cli.core import fab_constant, fab_logger, fab_state_config
from fabric_cli.core import fab_constant, fab_logger
from fabric_cli.core.fab_auth import FabAuth
from fabric_cli.core.fab_context import Context
from fabric_cli.core.fab_exceptions import FabricCLIError
Expand Down Expand Up @@ -262,21 +262,25 @@ def __mask_token(scope):
azure_secret = __mask_token(fab_constant.SCOPE_AZURE_DEFAULT)

# Check login status
is_logged_in = fabric_secret != "N/A"
login_status = (
Comment thread
Guust-Franssens marked this conversation as resolved.
"✓ Logged in to app.fabric.microsoft.com"
if fabric_secret != "N/A"
if is_logged_in
else "✗ Not logged in to app.fabric.microsoft.com"
)

fab_ui.print_grey(
f"""{login_status}
- Account: {upn} ({oid})
- Tenant ID: {tid}
- App ID: {appid}
- Token (fabric/powerbi): {fabric_secret}
- Token (storage): {storage_secret}
- Token (azure): {azure_secret}"""
)
fab_ui.print_grey(login_status)

auth_data = {
"logged_in": is_logged_in,
"account": upn,
"principal_id": oid,
"tenant_id": tid,
"app_id": appid,
"token_fabric_powerbi": fabric_secret,
"token_storage": storage_secret,
"token_azure": azure_secret,
}
fab_ui.print_output_format(args, data=auth_data, show_key_value_list=True)


# Utils
Expand Down
Loading