`suborgproperties` with a value that contains spaces matches no repositories
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 80/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- github, javascript
- Área
- backend
Línea de trabajo
Empieza en lib/settings.js, en getRepositoriesByProperty, donde, según el issue, la consulta de repositorios se construye sin poner entre comillas los valores de las propiedades. Compara esa ruta de búsqueda con repoMatchesProperties y con el comportamiento de sincronización de un solo repositorio descrito en el issue. Se considerará terminado cuando los valores con espacios encuentren los mismos repositorios en la sincronización completa y en la sincronización de un solo repositorio; verifícalo con una ejecución en seco usando el ejemplo proporcionado de una propiedad de suborg.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Problem Description
What is actually happening
A suborg file that matches on a custom property value with spaces matches no repositories on a full sync or a suborg file change. There's no error or warning, so the suborg is silently ignored.
suborgproperties:
- commit_message_rule_selector: DEFAULT - all branches
getRepositoriesByProperty in lib/settings.js builds the query as props.${name}:${value} without quotes, so the search reads the value as separate terms:
props.commit_message_rule_selector:DEFAULT - all branchesreturns no repositoriesprops.commit_message_rule_selector:"DEFAULT - all branches"returns the expected repositoriesprops.required_checks_profile:strict(no spaces) works as expected
When a single repository is synced, getSubOrgConfigsForRepo uses repoMatchesProperties, which compares values exactly. So the same suborg can match a repository on a single-repo sync but not on a full sync.
What is the expected behavior
Values with spaces match the same repositories on every sync path, for example by quoting the value in the repository_query.
Error output, if available
None. The dry run shows no changes and no errors for the suborg.
Context
Are you using the hosted instance of probot/settings or running your own?
Running our own.
If running your own instance, are you using it with github.com or GitHub Enterprise?
github.com (GitHub Enterprise Cloud)
Version of probot/settings
3.0.0
- Lenguaje dominante
- JavaScript
- Estrellas
- 927
- Forks
- 234
- Merge medio
- 2 d 15 h
- PR fusionados (30 d)
- 11
Preparar el entorno
Inicia el contenedor de desarrollo del proyecto en tu navegador, con tu propia cuenta de GitHub.
- Incluye un Dockerfile o un archivo de Docker Compose
- Sin plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de github-community-projects/safe-settings
-
app_installations reports an error on every repo or suborg change when the plugin is not configuredAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 64/100
github-community-projects/safe-settings#1111 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
Repos transferred into the org are not synced (no repository.transferred handler)Posiblemente ocupada @breymander la tomó hace 16 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
github-community-projects/safe-settings#1107 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dry run only inspects the first 30 changed files (pulls.listFiles is not paginated)Posiblemente ocupada @3rwww1 la tomó hace 39 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
github-community-projects/safe-settings#1080 ·
Los mantenedores suelen responder en 1 día
-
PR comment diff table shows an empty vertical scrollbar due to malformed <tr> markupQuizá libre de nuevo Un pull request para esta issue se cerró sin fusionarse. Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
github-community-projects/safe-settings#1077 ·
Los mantenedores suelen responder en 1 día
-
Bug Report: environment and repo variables API calls don't paginate, failing with more than 10 itemsPosiblemente ocupada @cdav la tomó hace 91 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
github-community-projects/safe-settings#1040 ·
Los mantenedores suelen responder en 1 día
Todos los issues de github-community-projects/safe-settings
Issues similares
-
ai-driven-fuzzing bug claude Minor
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
linagora/cozy-client#1777 · 1 comentario · 1 reacción ·
Los mantenedores suelen responder en 1 día
-
[guide] AGENTS.md claims the coverage gate, e2e and full lint run on every PR after ADR 0011 focused validationPosiblemente ocupada @hivecommons-hive la tomó hoy. Abiertoagent/guide documentation hive/claimed-by-guide stage/040-covered-by-pr
Dificultad 2/5 1-3 horas Aptitud para principiantes 81/100
cncf/endusers#2006 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
check:passed feeds:add
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
Los mantenedores suelen responder en 4 días
-
CMS video element: "CmsVideoPlugin" sets `volume = 0`, so videos cannot be unmuted on Android ChromeAbiertoneeds-triage Platform(Default)
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
all-contributors/cli#587 ·
Los mantenedores suelen responder en 1 día