Empty server.oidc.adminRole/userRole silently enables RBAC instead of authentication-only mode
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 3/5
- Temps estimé
- 1-2 jours
- Accessibilité débutants
- 68/100
- Type d'issue
- Bug
- Clarté
- Plutôt claire
- Activité
- Active
- Stack technique
- helm, kubernetes, rust, yaml
- Domaine
- authentication, devops, infrastructure
Piste de recherche
Commencez par values.yaml et le template Helm qui émet conditionnellement admin_role et user_role, puis examinez OidcConfig et son comportement par défaut pour les rôles. Effectuez le rendu ou installez le chart avec les valeurs des deux rôles vides et examinez le ConfigMap généré. C’est terminé lorsque la documentation correspond au comportement et que le chart fournit une méthode de déploiement vérifiée uniquement pour l’authentification.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
User Story
As a cluster operator, I want to deploy OpenShell gateway in auth-only mode.
Problem Statement
values.yaml documents authentication-only mode thusly:
# -- Role name for admin access. Leave empty (with userRole also empty) for
# authentication-only mode. Both must be set or both empty.
adminRole: ""
# -- Role name for standard user access.
userRole: ""
But the chart guards both keys on truthiness:
{{- if .Values.server.oidc.adminRole }}
admin_role = {{ .Values.server.oidc.adminRole | quote }}
{{- end }}
so empty strings are omitted from the rendered TOML rather than emitted as "". OidcConfig then applies #[serde(default = "default_admin_role")], restoring openshell-admin / openshell-user. The gateway starts in RBAC mode with the stock role names.
Impact / Why This Matters
The chart provides no way to deploy in auth-only mode and its documentation is misleading.
Acceptance Criteria
- Documentation is changed to reflect actual chart behavior.
- Method to deploy in auth-only mode is added to the chart.
Reproduction Steps
- Install the chart with
server.oidc.issuerset andserver.oidc.adminRole/server.oidc.userRoleleft at their""defaults. kubectl -n openshell get cm openshell-config -o yaml— the[openshell.gateway.oidc]table contains noadmin_roleoruser_rolekey.
Environment
- OpenShell: 0.0.116 (gateway image and Helm chart)
- OS: Talos Linux v1.13.7
- Runtime: Kubernetes v1.36.2
- Deployment: See https://github.com/jfroy/flatops/blob/main/kubernetes/apps/openshell/openshell/app/helmrelease.yaml
- Langage dominant
- Rust
- Étoiles
- 8.7k
- Forks
- 1.3k
- Merge moyen
- 2 j 6 h
- PR mergées (30 j)
- 297
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de NVIDIA/OpenShell
-
area:docs
Difficulté 1/5 Moins d'une heure Accessibilité débutants 88/100
-
state:triage-needed
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
-
area:cli state:validated
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
-
state:triage-needed
Difficulté 1/5 Moins d'une heure Accessibilité débutants 90/100
-
area:build spike state:review-ready state:stale
Difficulté 2/5 Une demi-journée Accessibilité débutants 68/100
Toutes les issues de NVIDIA/OpenShell
Issues similaires
-
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
-
state:needs triage
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
zed-industries/zed#64680 · 2 commentaires ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
-
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
RustPython/RustPython#8802 ·
-
Difficulté 2/5 1-3 heures Accessibilité débutants 75/100
TheLarkInn/aipm#2390 ·