Build the User-Agent header from the assembly version
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 88/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- csharp
- Área
- backend-api-design, testing
Línea de trabajo
Lee QuickbaseNet/Services/QuickbaseClient.cs alrededor de la constante UserAgent y del constructor, y luego QuickbaseNet.UnitTests/Tests/QuickbaseClientTests.cs. Sustituye la versión codificada por la versión del ensamblado según lo especificado y añade una prueba que inspeccione el encabezado User-Agent predeterminado con un cliente nuevo. Ejecuta dotnet build y dotnet test --filter FullyQualifiedName~QuickbaseClientTests; la tarea estará terminada cuando se compilen los cinco frameworks de destino y pase la prueba del encabezado.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
QuickbaseClient sends User-Agent: QuickbaseNet/1.0.1 on every request. The package is at 1.0.2, and release versions come from the git tag, so the hardcoded string goes stale with every release.
What to do
- Replace the
UserAgentconstant with a value built from the assembly version:"QuickbaseNet/" + typeof(QuickbaseClient).Assembly.GetName().Version.ToString(3).ToString(3)gives1.0.2instead of1.0.2.0, and it builds on every target, including netstandard2.0 and net48. - Add a test to
QuickbaseClientTeststhat creates a client and checksclient.Client.DefaultRequestHeaders.UserAgent.ToString(). It reads the header straight off theHttpClient, so no request or mock is needed. Use a new client, not the shared_client: that one replacesClient, which drops the headers set in the constructor.
Where to look
QuickbaseNet/Services/QuickbaseClient.cs, lines 18 and 46QuickbaseNet.UnitTests/Tests/QuickbaseClientTests.cs
Done when
- No version number is hardcoded in
QuickbaseClient.cs - The new test asserts the header is
QuickbaseNet/plus the assembly's three-part version, and passes -
dotnet buildsucceeds for all five target frameworks
Testing it
dotnet build
dotnet test --filter FullyQualifiedName~QuickbaseClientTests
- Lenguaje dominante
- C#
- Estrellas
- 3
- Forks
- 2
- Merge medio
- 3 h 14 min
- PR fusionados (30 d)
- 2
Preparar el entorno
- Sin Dockerfile ni 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 hazeliscoding/quickbase-net
-
good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
-
good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
-
Add `Skip()` and `Top()` to `QuickbaseQueryBuilder`Posiblemente ocupada @bercedev la tomó hace 2 días. Abiertoenhancement good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
hazeliscoding/quickbase-net#16 · 1 comentario ·
-
Update the package metadata that still points at ducksoopPosiblemente ocupada @hazeliscoding la tomó hace 3 días. Abierto
hazeliscoding/quickbase-net#15 · 1 asignado ·
Todos los issues de hazeliscoding/quickbase-net
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Nimblesite/SharpLsp#468 ·
Los mantenedores suelen responder en 1 día
-
V4 alert-rule reads have no scope gate, so any authenticated credential can list alert rulesAbiertoalerts bug effort:S P2
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
nightscout/nocturne#2511 ·
Los mantenedores suelen responder en 1 día
-
good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
feature-request
Dificultad 2/5 1-3 horas Aptitud para principiantes 77/100
stripe/stripe-dotnet#3484 ·
Los mantenedores suelen responder en 2 días
-
bug mcp misleading small
Dificultad 2/5 1-3 horas Aptitud para principiantes 77/100
Los mantenedores suelen responder en 1 día