googleapis / googleapis/google-api-nodejs-client

Sheets `clear` operation breaks with HTTP/2

Ouverte
#3,602 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub
Langage dominant
TypeScript
Étoiles
12.2k
Forks
2k
Merge moyen
1 j 9 h
PR mergées (30 j)
24

Description

### Please make sure you have searched for information in the following guides.

- [x] Search the issues already opened: https://github.com/GoogleCloudPlatform/google-cloud-node/issues
- [x] Search StackOverflow: http://stackoverflow.com/questions/tagged/google-cloud-platform+node.js
- [x] Check our Troubleshooting guide: https://github.com/googleapis/google-cloud-node/blob/main/docs/troubleshooting.md
- [x] Check our FAQ: https://github.com/googleapis/google-cloud-node/blob/main/docs/faq.md
- [x] Check our libraries HOW-TO: https://github.com/googleapis/gax-nodejs/blob/main/client-libraries.md
- [x] Check out our authentication guide: https://github.com/googleapis/google-auth-library-nodejs
- [x] Check out handwritten samples for many of our APIs: https://github.com/GoogleCloudPlatform/nodejs-docs-samples

### A screenshot that you have tested with "Try this API".

N/A

### Link to the code that reproduces this issue. A link to a **public** Github Repository or gist with a minimal reproduction.

https://gist.github.com/neil-lobo/e632aae751fe7aec8bbc29a42c585015

### A step-by-step description of how to reproduce the issue, based on the linked reproduction.

1. Save `credentials.json` file with correct access to sheet to root of project
2. Replace `[sheet ID]` with accessible sheet
3. Run script
4. Observe output
5. Turn off http2
6. Observe output
### A clear and concise description of what the bug is, and what you expected to happen.

Using the http2 option, the clear operation does not work. I assume https://github.com/googleapis/google-api-nodejs-client/issues/1801#issuecomment-524921412 is no longer the case since there is an explicit option to enable http2 now.

Referring to the gist, the expected output is
```
finish get 200
finish clear 200

```

however the output is
```
finish get 200

```

Disabling http2 produces the correct output. Note that this is not simply a response issue, the operation itself does not reflect in the specified sheet.

### A clear and concise description WHY you expect this behavior, i.e., was it a recent change, there is documentation that points to this behavior, etc. **

Breaks usage

Guide de contribution

Ouvrir le guide de contribution

Piste de recherche

Commencez par le gist public lié et reproduisez l’opération clear de Sheets avec HTTP/2 activé, puis comparez-la au comportement fonctionnel de HTTP/1.1. Suivez le chemin client utilisé par le gist et vérifiez la correction en observant une réponse clear réussie et en constatant que la sheet spécifiée est effectivement vidée.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
node.js, typescript
Domaine
api, networking
Type d'issue
Bug
Difficulté
3/5
Temps estimé
1-2 jours
Activité
À l'abandon
Clarté
Clairement spécifiée
Accessibilité débutants
45/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.