AddSingleton does not work as expected
Les mainteneurs répondent en général sous 2 jours
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 25/100
- Type d'issue
- Bug
- Clarté
- À clarifier
- Activité
- À l'abandon
- Stack technique
- csharp
- Domaine
- backend-api-design
Piste de recherche
Commencez par reproduire les enregistrements signalés de AddSingleton(type, instance) et de factory dans la configuration des services du SDK, en utilisant le type d’outil fourni et le chemin Activator.CreateInstance. Comparez si l’instance enregistrée ou la factory est appelée lors de la création de l’outil ; c’est terminé lorsque les deux formes d’enregistrement sont respectées au lieu de créer silencieusement une autre instance.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Describe the bug
When we register tool like AddSingleton(type, instance) and pass a singleton instance, it seems that the instance is being ignored and the library creates a new instance when it needs it.
I tried also AddSingleton using a factory method like this:
services.AddSingleton(toolType, provider => Activator.CreateInstance(toolType, this.m_compositionService));
A breakpoint to Activator.CreateInstance was never hit and the tool just created the instance of the tool.
To Reproduce
Steps to reproduce the behavior:
1.
Expected behavior
A clear and concise description of what you expected to happen.
Logs
If applicable, add logs to help explain your problem.
Additional context
We discussed the issue over email. Please feel free to contact me there if you need additional information. Thanks!
- Langage dominant
- C#
- Étoiles
- 4.6k
- Forks
- 821
- Merge moyen
- 9 j 3 h
- PR mergées (30 j)
- 1
Préparer son environnement
Lance le conteneur de développement du projet dans votre navigateur, avec votre propre compte GitHub.
- 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 modelcontextprotocol/csharp-sdk
-
Reserved URI-template expansion throws for supplementary Unicode charactersPeut-être pris @pournasserian l’a pris il y a 1 jour. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 74/100
modelcontextprotocol/csharp-sdk#1913 · 1 commentaire ·
Les mainteneurs répondent en général sous 2 jours
-
ClaimsPrincipal parameter injection sample requestPeut-être pris @utsavjain2809 l’a pris il y a 11 jours. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 74/100
modelcontextprotocol/csharp-sdk#1899 ·
Les mainteneurs répondent en général sous 2 jours
-
Document ClientOAuthOptions.ScopeSelector usagePeut-être pris @z0rimo l’a pris il y a 34 jours. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
modelcontextprotocol/csharp-sdk#1867 · 1 commentaire ·
Les mainteneurs répondent en général sous 2 jours
-
HTTP+SSE client: POST responses are never disposed, leaking one connection per sent messagePeut-être pris @yalcinfu22 l’a pris il y a 49 jours. Ouverte
Difficulté 1/5 Moins d'une heure Accessibilité débutants 88/100
modelcontextprotocol/csharp-sdk#1840 · 1 commentaire ·
Les mainteneurs répondent en général sous 2 jours
-
enhancement needs confirmation
Difficulté 2/5 1-3 heures Accessibilité débutants 64/100
modelcontextprotocol/csharp-sdk#678 · 1 commentaire ·
Les mainteneurs répondent en général sous 2 jours
Toutes les issues de modelcontextprotocol/csharp-sdk
Issues similaires
-
documentation good first issue
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
Les mainteneurs répondent en général sous 1 jour
-
[Bug]: Edit Bar buttons are not localizedPeut-être pris @idumlupinar l’a pris aujourd’hui. OuverteStatus: New
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
dnnsoftware/Dnn.Platform#7565 ·
Les mainteneurs répondent en général sous 1 jour
-
AzureBlobSnapshotStore.LoadAsync allocates the snapshot twice (growable MemoryStream + ToArray)Ouverteenhancement
Difficulté 2/5 Une demi-journée Accessibilité débutants 72/100
-
bug ui-only
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
Les mainteneurs répondent en général sous 1 jour
-
bug
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
ClickHouse/clickhouse-cs#689 ·
Les mainteneurs répondent en général sous 2 jours