[PR #440] [MERGED] Handled all curl error without exiting process - #437 #1507

New issue

Closed

opened 2026-03-04 02:00:48 +03:00 by kerem · 0 comments

kerem commented

2026-03-04 02:00:48 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/s3fs-fuse/s3fs-fuse/pull/440
Author: @ggtakec
Created: 6/27/2016
Status: ✅ Merged
Merged: 6/27/2016
Merged by: @ggtakec

Base: master ← Head: master

📝 Commits (1)

6005929 Handled all curl error without exiting process - #437

📊 Changes

1 file changed (+7 additions, -7 deletions)

View changed files

📝 src/curl.cpp (+7 -7)

📄 Description

Changed S3fsCurl::RequestPerform() method.
All error result case returned curl_easy_perform() returns error code instead of process exit.
Those are CURLE_SSL_CACERT / CURLE_PEER_FAILED_VERIFICATION and unknown(default) cases.
This changes are a fix for #437.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/s3fs-fuse/s3fs-fuse/pull/440 **Author:** [@ggtakec](https://github.com/ggtakec) **Created:** 6/27/2016 **Status:** ✅ Merged **Merged:** 6/27/2016 **Merged by:** [@ggtakec](https://github.com/ggtakec) **Base:** `master` ← **Head:** `master` --- ### 📝 Commits (1) - [`6005929`](https://github.com/s3fs-fuse/s3fs-fuse/commit/6005929a96087e88bfb168d81cf9688bc49b7380) Handled all curl error without exiting process - #437 ### 📊 Changes **1 file changed** (+7 additions, -7 deletions) <details> <summary>View changed files</summary> 📝 `src/curl.cpp` (+7 -7) </details> ### 📄 Description Changed S3fsCurl::RequestPerform() method. All error result case returned curl_easy_perform() returns error code instead of process exit. Those are CURLE_SSL_CACERT / CURLE_PEER_FAILED_VERIFICATION and unknown(default) cases. This changes are a fix for #437. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>