User Namespaces and Common Methods project settings have no effect on stacking
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 3/5
- Tiempo estimado
- 1-2 días
- Aptitud para principiantes
- 68/100
Línea de trabajo
Read EventPipeline.cs and 20_ErrorPlugin.cs to follow how project settings reach the event context, then check the settings access patterns for @@DataExclusions, @@UserAgentBotPatterns and @@IncludePrivateInformation. Update the reads for UserNamespaces and CommonMethods with the stated fallback, and verify stacking for both configuration settings and existing data properties; consider whether migrating Project.Data is needed for the settings page.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Problem
The User Namespaces and Common Methods fields on the project settings page are saved, but the event pipeline never sees them, so they don't affect stacking.
- The UI saves both as project configuration settings:
settings/+page.svelte#L79and#L131callupdateOrRemoveProjectConfig(...). EventPipelineonly copies organization and projectDatainto the event context:EventPipeline.cs#L49-L58.ErrorPluginreads them from the context properties:20_ErrorPlugin.cs#L35-L43.
Reproduction
Verified against exceptionless/exceptionless:8.10.0 (StackingSignatureE2ETest):
POST /api/v2/projects/{id}/config?key=UserNamespaceswithcom.example(what the UI does). Submit an error thrown fromInteger.parseIntcalled bycom.example.Foo. Result: the signature method is stilljava.lang.NumberFormatException.forInputString(...).POST /api/v2/projects/{id}/data?key=UserNamespaceswithcom.example. Submit the same error. Result: the signature method iscom.example.Foo.parse(...)as expected.
Proposal
ErrorPlugin should read UserNamespaces and CommonMethods from context.Project.Configuration.Settings, falling back to the existing data properties for backwards compatibility. That's also how @@DataExclusions, @@UserAgentBotPatterns and @@IncludePrivateInformation are read.
Optionally migrate existing Project.Data values into the configuration so the settings page shows them.
Note: configuration settings are also sent to clients by /projects/config. That's harmless, since clients ignore keys they don't know. Hiding them from the client-configuration table is already handled (config-options.svelte.ts#L61).
- Lenguaje dominante
- C#
- Estrellas
- 2.5k
- Forks
- 508
- Merge medio
- 2 d 17 h
- PR fusionados (30 d)
- 40
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 exceptionless/Exceptionless
-
bug documentation
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
exceptionless/Exceptionless#2707 ·
Los mantenedores suelen responder en 1 día
-
enhancement
Dificultad 2/5 1-3 horas Aptitud para principiantes 85/100
exceptionless/Exceptionless#2706 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
exceptionless/Exceptionless#2705 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
exceptionless/Exceptionless#2704 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 73/100
exceptionless/Exceptionless#2703 ·
Los mantenedores suelen responder en 1 día
Todos los issues de exceptionless/Exceptionless
Issues similares
-
Add 'upm' tag on githubAbierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 85/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Los mantenedores suelen responder en 1 día
-
bug mac アプリ: コード アプリ: チャット 優先: 中 要トリアージ
Dificultad 2/5 1-3 horas Aptitud para principiantes 74/100
yksr-melt/Meltype#1346 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
Collect corrected translations in the app, with no GitHub account and the least possible frictionAbiertolocalization medium ui
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
hexide-io/HexIDE#2819 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100