-
Notifications
You must be signed in to change notification settings - Fork 4.4k
Structured Error Output #9890
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.
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
Open
AndrewAsseily
wants to merge
32
commits into
aws:feature/str-std-error
Choose a base branch
from
AndrewAsseily:nyandrew/structured-error
base: feature/str-std-error
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Structured Error Output #9890
Changes from all commits
Commits
Show all changes
32 commits
Select commit
Hold shift + click to select a range
1afe452
Structured errors draft - failing tests
AndrewAsseily c46cf3e
Merge branch 'v2' into nyandrew/structured-error
AndrewAsseily 6842e4e
Revert "Structured errors draft - failing tests"
AndrewAsseily 3fe5630
Merge branch 'v2' into nyandrew/structured-error
AndrewAsseily bcc33fc
Structured Errors implementation Draft 1
AndrewAsseily 829968b
Refactor structured error
AndrewAsseily 181b94a
Merge branch 'v2' into nyandrew/structured-error
AndrewAsseily 6958d85
Remove cli_hide_error_details
AndrewAsseily 9eef83f
Merge branch 'v2' into nyandrew/structured-error
AndrewAsseily 329d167
Implement --output off format to suppress stdout
AndrewAsseily 5700443
Merge branch 'v2' into nyandrew/structured-error
AndrewAsseily 073c28d
Add missing off status to output choices
AndrewAsseily 60b04ee
Merge branch 'v2' into nyandrew/structured-error
AndrewAsseily b82ef33
Catch and format ClientError exceptions raised during pagination
AndrewAsseily 4b34b62
Merge branch 'v2' into nyandrew/structured-error
AndrewAsseily 7f511a2
Update test fixtures and documentation for 'off' output format option
AndrewAsseily ae6987d
Remove TODO for filtering; use --output off for sensitive data instead
AndrewAsseily 9db425e
Add changelog entry for structured error output
AndrewAsseily bc25515
Moves structured error handling into ClientErrorHandler to write erro…
AndrewAsseily ce86991
Remove cli_error_format from global options docs (config-only variable)
AndrewAsseily 29ee23c
Add structured error output with configurable formats (json, yaml, en…
AndrewAsseily e003f09
Update changelog entry to reflect new error format settings
AndrewAsseily 4f87cfa
Update validation text and format relevant files
AndrewAsseily 64a79eb
Refactor error handling: simplify logic, add constants
AndrewAsseily a7548f6
Update documentation to include --cli-error-format global option
AndrewAsseily 3c14ea4
Fix legacy format test to verify modeled fields are not displayed
AndrewAsseily 408c0c6
Fix error handler double construction and improve error format messaging
AndrewAsseily 8c77925
Extract service-specific error fields from top-level response
AndrewAsseily 8a4c74b
Remove text format from complex value suggestions due to poor array h…
AndrewAsseily aa19e40
Simplified the format_error method to always use consistent formatting
AndrewAsseily b4db2ae
Add the new error format to the general options table in config-vars
AndrewAsseily 4232a97
Restore error handler fallback
AndrewAsseily File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,5 @@ | ||
| { | ||
| "type": "enhancement", | ||
| "category": "Output", | ||
| "description": "Add support for ``--output off`` to suppress all stdout output while preserving stderr for errors and warnings." | ||
| } |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,5 @@ | ||
| { | ||
| "type": "feature", | ||
| "category": "Output", | ||
| "description": "Add structured error output with configurable formats. AWS service errors now display additional fields in the configured format (legacy, json, yaml, text, table, or enhanced). Configure via ``--cli-error-format``, ``cli_error_format`` config variable, or ``AWS_CLI_ERROR_FORMAT`` environment variable. The new enhanced format is the default. Set ``cli_error_format=legacy`` to preserve the original error format." | ||
| } |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -16,3 +16,4 @@ | |
| [--no-cli-pager] | ||
| [--cli-auto-prompt] | ||
| [--no-cli-auto-prompt] | ||
| [--cli-error-format <value>] | ||
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.