feat: add the ability to specify global and specific parameters through the config ini file

[dnitsch]

---

name:

Describe the PR
added more sections to the config file

fix: add more tests

rework how the config file is passed in

perform merges from config.specific sections to the global config

Screenshots
If applicable, add screenshots to help explain your problem.

Additional context
Add any other context about the problem here.

+semver: feature
+semver: FEATURE

[Copilot]

Pull Request Overview

This PR adds the ability to specify global and specific parameters through a config INI file, enhancing the configuration management system.

Key changes include:

• Introduces configuration file parsing with global [config] sections and specific [config.specific_section] sections
• Adds support for merging configuration from files with command-line flags
• Refactors method signatures to pass configuration data explicitly rather than loading it internally

Reviewed Changes

Copilot reviewed 17 out of 18 changed files in this pull request and generated 4 comments.

Show a summary per file

File	Description
internal/web/web.go	Adds method to set custom browser executable path
internal/credentialexchange/secret.go	Refactors ClearAll method to accept INI file parameter and moves method positioning
internal/credentialexchange/helper.go	Adds LoadCliConfig function for merging global and specific configuration sections
internal/credentialexchange/config.go	Adds INI tags to struct fields for configuration mapping
cmd/saml.go	Major refactoring to support configuration file loading and flag merging
cmd/awscliauth.go	Updates root command flags structure and adds config file location parameter
cmd/clear.go	Updates to use new ClearAll method signature

Comments suppressed due to low confidence (1)

internal/credentialexchange/helper_test.go:1

• All t.Error() calls should include descriptive error messages explaining what was expected vs actual values to aid in debugging test failures.

package credentialexchange_test

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

[Copilot]

Error from os.MkdirTemp should be checked and handled. The underscore assignment ignores potential errors that could cause test failures.

Suggested change

	tmpDir, _ := os.MkdirTemp(os.TempDir(), "saml-cred-test-*")
	tmpDir, err := os.MkdirTemp(os.TempDir(), "saml-cred-test-*")
	if err != nil {
	t.Fatal(err)
	}

[sonarqubecloud]

Quality Gate passed

Issues
2 New issues
0 Accepted issues

Measures
0 Security Hotspots
87.7% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud