[PR #192] [MERGED] Simplify URL encoding #1390

New issue

Closed

opened 2026-03-04 01:53:42 +03:00 by kerem · 0 comments

kerem commented

2026-03-04 01:53:42 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/s3fs-fuse/s3fs-fuse/pull/192
Author: @gaul
Created: 6/6/2015
Status: ✅ Merged
Merged: 6/20/2015
Merged by: @ggtakec

Base: master ← Head: special-characters

📝 Commits (1)

4e03acf Simplify URL encoding

📊 Changes

2 files changed (+37 additions, -28 deletions)

View changed files

📝 src/string_util.cpp (+26 -28)
📝 test/integration-test-main.sh (+11 -0)

📄 Description

This also encodes asterisk and tilde correctly when listing a file
with a V4 auth endpoint. Also add tests for special characters
although s3proxy does not yet support V4 auth.
Fixes #188. Fixes #194.

_{🔄 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/192 **Author:** [@gaul](https://github.com/gaul) **Created:** 6/6/2015 **Status:** ✅ Merged **Merged:** 6/20/2015 **Merged by:** [@ggtakec](https://github.com/ggtakec) **Base:** `master` ← **Head:** `special-characters` --- ### 📝 Commits (1) - [`4e03acf`](https://github.com/s3fs-fuse/s3fs-fuse/commit/4e03acf17ad825458aa435c0f3fed9badfbb0a05) Simplify URL encoding ### 📊 Changes **2 files changed** (+37 additions, -28 deletions) <details> <summary>View changed files</summary> 📝 `src/string_util.cpp` (+26 -28) 📝 `test/integration-test-main.sh` (+11 -0) </details> ### 📄 Description This also encodes asterisk and tilde correctly when listing a file with a V4 auth endpoint. Also add tests for special characters although s3proxy does not yet support V4 auth. Fixes #188. Fixes #194. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>