Ji-Xinyou commented on PR #2190:
URL: 
https://github.com/apache/incubator-opendal/pull/2190#issuecomment-1531668063

   > Maybe we just need to make sure the path has been percent encoded?
   
   Following the first line is what we are using in our behavior tests. As I 
tested, the third line which removed '#', '%' and '^' works out fine.
   
   ```Rust
   let path = format!("{} !@#$%^&()_+-=;',.txt", uuid::Uuid::new_v4());
   // remove # % and ^
   let path = format!("{} !@$&()_+-=;',.txt", uuid::Uuid::new_v4());
   ```
   
   In the `Characters to avoid` section from the 
[this](https://docs.aws.amazon.com/AmazonS3/latest/userguide/object-keys.html), 
we can see `Caret ("^")`, `Percent character ("%")` and `'Pound' character 
("#")` should be avoided.
   
   Maybe we should remove these three characters, at least from the amazon's 
spec we should avoid these three.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to