[PR #786] [MERGED] Log messages for 5xx and 4xx HTTP response code #1639

New issue

Closed

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

kerem commented

2026-03-04 02:01:26 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/s3fs-fuse/s3fs-fuse/pull/786
Author: @ambiknai
Created: 7/4/2018
Status: ✅ Merged
Merged: 7/8/2018
Merged by: @ggtakec

Base: master ← Head: log_enhancements

📝 Commits (4)

92a4034 Log messages for 5xx and 4xx HTTP response code
ad5349a Changes as per review comments
4177d8b Review comment: Include the error code being returned
35d3fce Review comment: Include the error code being returned

📊 Changes

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

View changed files

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

📄 Description

Relevant Issue (if applicable)

If there are Issues related to this PullRequest, please list it.

Details

Currently S3FS fuse is not logging any message for 4xx and 5xx HTTP response codes.
The code change enable the logging for 4xx and 5xx codes.

_{🔄 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/786 **Author:** [@ambiknai](https://github.com/ambiknai) **Created:** 7/4/2018 **Status:** ✅ Merged **Merged:** 7/8/2018 **Merged by:** [@ggtakec](https://github.com/ggtakec) **Base:** `master` ← **Head:** `log_enhancements` --- ### 📝 Commits (4) - [`92a4034`](https://github.com/s3fs-fuse/s3fs-fuse/commit/92a4034c5e377aeed273c5c3f2f6a3c931b9e93e) Log messages for 5xx and 4xx HTTP response code - [`ad5349a`](https://github.com/s3fs-fuse/s3fs-fuse/commit/ad5349a488e6045c8b89edd1514f31145a8223a6) Changes as per review comments - [`4177d8b`](https://github.com/s3fs-fuse/s3fs-fuse/commit/4177d8bd3bb6776258f70fb74d92c6cc243358c9) Review comment: Include the error code being returned - [`35d3fce`](https://github.com/s3fs-fuse/s3fs-fuse/commit/35d3fce7a03e89c43c409689e69a7643e2c52585) Review comment: Include the error code being returned ### 📊 Changes **1 file changed** (+4 additions, -7 deletions) <details> <summary>View changed files</summary> 📝 `src/curl.cpp` (+4 -7) </details> ### 📄 Description ### Relevant Issue (if applicable) If there are Issues related to this PullRequest, please list it. ### Details Currently S3FS fuse is not logging any message for 4xx and 5xx HTTP response codes. The code change enable the logging for 4xx and 5xx codes. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>