Skip to content
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

Add Profile Name and Line Number to Config Save Error Messages #343

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

brandonkgarner
Copy link

Adds profile name and line number to error messages during configuration save in the format: [profile_name]:##, or in the case that the profile name is not in the line (such as invalid parameter definition), grabs the value of the line: [line_value]:##

Addresses #338 as I have run into the same issue multiple times.

Affects only ConfigLoadError/IniParseError that already supply a line-number from aesr-config:

  • "The profile includes both role_arn and either aws_account_id or role_name."
  • "The profile includes invalid role_arn parameter."
  • "The profile doesn't specify an AWS account ID."
  • "Invalid parameter definition"

Tested in Chrome and Firefox on MacOS 14.4.1 with the same results.
chrome-invalid_role_arn
chrome-aws_account_not_specified
chrome-invalid_param
chrome-conflicting_options

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant