Why aren't RestClient::RequestTimeout exceptions caught and re-raised as CloudinaryExceptions?
@gtmax arbeitet bereits daran.
Seit 23.3.2017.
Bewertung
Dieses Issue wurde noch nicht bewertet.
Beschreibung
The call_api method only raises CloudinaryExceptions under certain circumstances, like JSON parse errors or certain failure codes. But if the underlying RestClient times out, it's not caught so it bubbles out to the calling code.
https://github.com/cloudinary/cloudinary_gem/blob/master/lib/cloudinary/uploader.rb#L293-L308
Is this an oversight or intentional?
It's nice to only think about the Cloudinary API when coding and not have to worry about libraries that Cloudinary uses (like RestClient).
I'd be happy to submit a PR on this, but I wanted to start with a question to see if I'm not understanding the rationale here. Thanks!
- Vorherrschende Sprache
- Ruby
- Sterne
- 420
- Forks
- 282
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Hat eine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus cloudinary/cloudinary_gem
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 55/100
cloudinary/cloudinary_gem#597 · 1 Kommentar ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 52/100
cloudinary/cloudinary_gem#591 · 3 Kommentare ·
-
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 45/100
cloudinary/cloudinary_gem#582 · 1 Kommentar ·
-
404's when fetching .eml files (from Action Mailbox)Evtl. wieder frei @rnamba-cloudinary hat das vor 400 Tagen übernommen, und es ist kein Pull Request offen. Offen
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 30/100
cloudinary/cloudinary_gem#581 · 1 Kommentar ·
-
BlobKey, MirrorService and Sidekiq 8 are incompatibleEvtl. wieder frei @const-cloudinary hat das vor 541 Tagen übernommen, und es ist kein Pull Request offen. Offen
cloudinary/cloudinary_gem#573 · 3 Kommentare · 1 zugewiesene Person ·
Alle Issues in cloudinary/cloudinary_gem
Ähnliche Issues
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100
metacall/core#964 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
area: ci benchmark build developer-experience difficulty: easy dx enhancement github-actions good first issue hacktoberfest help wanted performance quality rank: 8 kyu ruby tech: ruby
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 82/100
flisk-sh/flisk-labs#22 ·
-
1.severity: security
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 62/100
Maintainer antworten meist innerhalb von 1 Tag
-
15-minutes 5-minutes algorithm any-language beginner-friendly c cpp csharp easy go good first issue hacktoberfest help wanted java javascript kotlin python rust swift typescript
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 68/100
Huerte/algo-solutions-python#260 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 72/100
Maintainer antworten meist innerhalb von 1 Tag