9001 / 9001/copyparty

idp groupnames cannot contain `:`

Ouverte
#1,461 0 commentaires 0 réactions 1 personne assignée Réclamée par @9001 Voir sur GitHub
bug
Langage dominant
Python
Étoiles
46.7k
Forks
1.9k
Merge moyen
9 j 2 h
PR mergées (30 j)
4

Description

given an idp-group with the literal name `GRP:foo` as used in the below config, the server will crash on startup:

```yaml
[/]
srv
accs:
A: @GRP:foo
```

**workaround:** until this is fixed, remap `:` to `/` or something else;

```yaml
[global]
idp-chsub: |:/|

[/]
srv
accs:
A: @GRP/foo
```

the crash is an errormessage complaining that the config syntax is invalid due to an exception from the following sanchk: https://github.com/9001/copyparty/blob/da6e2ddca96dffc4dbe53bda25d2034428fad3d0/copyparty/authsrv.py#L1521-L1522

Guide de contribution

Ouvrir le guide de contribution

Piste de recherche

Le bug se trouve dans copyparty/authsrv.py aux lignes 1521-1522, où une vérification de cohérence échoue pour les noms de groupe contenant des deux-points. Commence par lire la logique d’analyse de la configuration et de validation des noms de groupe autour de cette zone. Teste en créant une configuration avec un groupe comme 'GRP:foo' et en lançant le serveur pour observer le plantage. La correction consiste probablement à ajuster l’expression régulière ou la validation afin d’autoriser les deux-points dans les noms de groupe, puis à vérifier que le contournement avec idp-chsub fonctionne toujours.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
python
Domaine
authentication, backend
Type d'issue
Bug
Difficulté
3/5
Temps estimé
1-2 jours
Activité
À l'abandon
Clarté
Clairement spécifiée
Accessibilité débutants
55/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.