Feature/provider command#1
Open
aaron-riact wants to merge 77 commits into
Open
Conversation
326ede0 to
6a37da5
Compare
There was a problem hiding this comment.
Pull Request Overview
This PR introduces client configuration management commands to enable users to interact with provider configurations through a CLI interface.
- Adds new
clientcommand with subcommands for listing providers and initializing configuration - Refactors client configuration handling into reusable utility functions
- Adds terminal detection for improved output formatting
Reviewed Changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| main.go | Adds client command structure with provider list and init-config subcommands |
| go.mod | Adds golang.org/x/term dependency for terminal detection |
| flake.nix | Updates vendor hash for new dependencies |
| commands/login.go | Refactors to use new client config utility functions |
| commands/config/client_config.go | Adds utility functions for config path resolution and file operations |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
9143ce3 to
9c29d17
Compare
ec315bd to
88a24f1
Compare
* fix(deps): Update Docker * Fix broken ubuntu docker image --------- Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: Ethan Heilman <ethan.r.heilman@gmail.com>
* extract utility functions * add client provider list command * remove unused param * extract config path resolution * prefer plain old tabwriter * no headers or tables if no tty * preserve provider order * add client provider list test * check map construction after reporting to user * update nix flake vendorHash Signed-off-by: Ethan Heilman <ethan.r.heilman@gmail.com> --------- Signed-off-by: Ethan Heilman <ethan.r.heilman@gmail.com> Co-authored-by: Mark Farrell <mark.andrew.farrell@gmail.com>
…ey#308) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: Ethan Heilman <ethan.r.heilman@gmail.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: Ethan Heilman <ethan.r.heilman@gmail.com>
Fixes URL in get_te_download_path Signed-off-by: Jonas Forsberg <barregargamel@gmail.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: Ethan Heilman <ethan.r.heilman@gmail.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
* Merge SELinux Type Enforcement Files. * Merge the TE files in to one file and use SELinux booleans to enable and disable functions. * Adds SELinux documentation * Sets min version to v0.10.0 Signed-off-by: Jonas Forsberg <barregargamel@gmail.com> --------- Signed-off-by: Jonas Forsberg <barregargamel@gmail.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
* fix typo in commands/verify.go --------- Co-authored-by: Ethan Heilman <ethan.r.heilman@gmail.com>
* fix(deps): Update Docker * Fix openssl version conflict * Fix ubuntu image hash --------- Signed-off-by: Ethan Heilman <ethan.r.heilman@gmail.com> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: Ethan Heilman <ethan.r.heilman@gmail.com>
* Add support for custom group claims * Minor change, extra tests --------- Co-authored-by: Ethan Heilman <ethan.r.heilman@gmail.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
The last version of OSX went end of life almost 7 years ago. This updates references in the README to the current Apple macOS. Signed-off-by: Sean McGinnis <sean.mcginnis@gmail.com> Co-authored-by: Ethan Heilman <ethan.r.heilman@gmail.com>
…openpubkey#342) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
docs: fix typos Signed-off-by: Emanuel Guével <guevel.emanuel@gmail.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
…ey#335) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: Ethan Heilman <ethan.r.heilman@gmail.com>
…penpubkey#415) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
* Add documentation for opkssh and sssd integration Document the integration of opkssh with sssd for SSH login policies, including configuration steps and testing procedures. --------- Signed-off-by: Vignesh <62500639+vigneshmanick@users.noreply.github.com> Signed-off-by: Ethan Heilman <ethan.r.heilman@gmail.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
…ubkey#418) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
…ey#413) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: Ethan Heilman <ethan.r.heilman@gmail.com>
* Add Amazon Cognito as tested provider --------- Signed-off-by: Foorack / Max Faxälv <max@foorack.com>
…ubkey#426) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Fix integration tests by refreshing pacman keys
* Process extra arguments to the verify command * Add unit tests * Add docs for additional functionality
Co-authored-by: EthanHeilman <274814+EthanHeilman@users.noreply.github.com>
* Add Keycloak config docs * Add Keycloak to tested providers table in readme Signed-off-by: Sylvain Gable <sylvain@vingtcinq.io> --------- Signed-off-by: Sylvain Gable <sylvain@vingtcinq.io> Co-authored-by: Ethan Heilman <ethan.r.heilman@gmail.com>
Add a note about the behaviour of the azure alias
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Basti-Fantasti <1663072+Basti-Fantasti@users.noreply.github.com> Co-authored-by: Ethan Heilman <ethan.r.heilman@gmail.com>
Co-authored-by: EthanHeilman <274814+EthanHeilman@users.noreply.github.com>
* Remote Redirect URIs Support (termix) and unittests * Upgrading to 1.24 for ci builder
Co-authored-by: EthanHeilman <274814+EthanHeilman@users.noreply.github.com>
0b7db39 to
523f615
Compare
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
No description provided.