New method for downloading streams
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Accessibilité débutants
- 35/100
Piste de recherche
Commencez par suivre la méthode Download existante et le point d’entrée ExecuteStreamAsync proposé, puis examinez RestResponse, RestResponseBase et RestStreamResponse, y compris le source generator terminé. C’est terminé lorsque les téléchargements en streaming exposent les erreurs de réponse et une disposable response avec Stream?, au lieu de renvoyer uniquement Task<Stream?>.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
My current thought is to change the interface signature and make Download work differently. Right now, it returns Task<Stream?>, which prevents the user to inspect the response for errors. It crashes if the call wasn't successful.
So, the idea is to make ExecuteStreamAsync, which will return Task<RestStreamResponse>. The response will be like RestResponse but instead of Content it will have Stream?. Naturally, it requires to be IDisposable.
The first step there was to create a source generator to create instance of RestResponseBase child class, as copying those properties manually is error prone (adding a new prop requires many changes). That part is done, so now this thing should be relatively easy.
Originally posted by @alexeyzimarev in https://github.com/restsharp/RestSharp/issues/2072#issuecomment-2174572685
- Langage dominant
- C#
- Étoiles
- 9.8k
- Forks
- 2.3k
- Métriques de merge des PR
- Aucune PR mergée en 30 j
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
- Propose un 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 restsharp/RestSharp
-
RestClientOptions.UserAgent missing from requests with useClientFactory: truePeut-être pris Une pull request liée à cette issue est ouverte ou déjà fusionnée. Ouvertebug
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
-
Expose HTTP response trailers on RestResponsePeut-être pris @Benziza l’a pris il y a 16 jours. Ouverte
Difficulté 4/5 3-5 jours Accessibilité débutants 68/100
-
RestClient configured with Newtonsoft.Json serializer still instantiates System.Text.JsonPeut-être pris @JiuYue0820 l’a pris il y a 58 jours. Ouvertebug
Difficulté 3/5 1-2 jours Accessibilité débutants 65/100
-
bug
Difficulté 4/5 3-5 jours Accessibilité débutants 48/100
-
feature-request rfc
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 25/100
Toutes les issues de restsharp/RestSharp
Issues similaires
-
documentation
Difficulté 1/5 1-3 heures Accessibilité débutants 62/100
-
bug misleading tooling trivial
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
Les mainteneurs répondent en général sous 1 jour
-
Docs example projects pinned to a literal library version instead of 3.*Peut-être pris @DaveSkender l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
facioquo/stock-indicators-dotnet#2372 ·
Les mainteneurs répondent en général sous 1 jour
-
[Docs] officecli-docx SKILL.md still describes `\n` as a soft line break after #262Peut-être pris @giprime l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 82/100
Les mainteneurs répondent en général sous 2 jours
-
Difficulté 2/5 1-3 heures Accessibilité débutants 62/100
Facepunch/sbox-public#12097 ·
Les mainteneurs répondent en général sous 2 jours