Build the User-Agent header from the assembly version
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 88/100
- Type d'issue
- Bug
- Clarté
- Clairement spécifiée
- Activité
- Active
- Stack technique
- csharp
- Domaine
- backend-api-design, testing
Piste de recherche
Lisez QuickbaseNet/Services/QuickbaseClient.cs autour de la constante UserAgent et du constructeur, puis QuickbaseNet.UnitTests/Tests/QuickbaseClientTests.cs. Remplacez la version codée en dur par la version de l’assembly comme indiqué et ajoutez un test qui vérifie l’en-tête User-Agent par défaut à l’aide d’un nouveau client. Exécutez dotnet build et dotnet test --filter FullyQualifiedName~QuickbaseClientTests ; le travail est terminé lorsque les cinq frameworks cibles sont compilés et que le test de l’en-tête réussit.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
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
- Langage dominant
- C#
- Étoiles
- 3
- Forks
- 2
- Merge moyen
- 3 h 14 min
- PR mergées (30 j)
- 2
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire 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 hazeliscoding/quickbase-net
-
good first issue
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
-
good first issue
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
-
Add `Skip()` and `Top()` to `QuickbaseQueryBuilder`Peut-être pris @bercedev l’a pris il y a 1 jour. Ouverteenhancement good first issue
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
hazeliscoding/quickbase-net#16 · 1 commentaire ·
-
Update the package metadata that still points at ducksoopPeut-être pris @hazeliscoding l’a pris il y a 2 jours. Ouverte
hazeliscoding/quickbase-net#15 · 1 personne assignée ·
Toutes les issues de hazeliscoding/quickbase-net
Issues similaires
-
[Bug]: Revit selector reads process ids 2000-2099 as years (flaky AgentClientTests.AYearOrAProcessIdSelectsTheHost)Peut-être pris @Wurschdhaud l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 83/100
pyrevitlabs/pyRevit#3817 ·
Les mainteneurs répondent en général sous 1 jour
-
bug effort:S P2
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
nightscout/nocturne#2449 ·
Les mainteneurs répondent en général sous 1 jour
-
audit:1 bug
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
NethermindEth/dotnet-libp2p#395 ·
Les mainteneurs répondent en général sous 1 jour
-
area: client bug
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
Les mainteneurs répondent en général sous 1 jour
-
bug mcp misleading small
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
Les mainteneurs répondent en général sous 1 jour