phreed opened a new issue, #6123: URL: https://github.com/apache/opendal/issues/6123
### Describe the bug https://github.com/apache/opendal/blob/v0.53.1/core/src/raw/http_util/client.rs#L164-L169 The expect message produced here is not ideal. Here is an example of its presentation. "input request url must be valid: InvalidIpv4Address" (see https://github.com/prefix-dev/rattler-build/issues/1595#event-17451538555) It would be helpful if the message contained the defective url. Maybe something like the following: ```rust reqwest::Url::from_str(&uri.to_string()).expect(format!("input request url must be valid {}", uri)), ``` ### Steps to Reproduce Provide a defective uri and observe that the defective uri is not presented. ### Expected Behavior I expect to have enough information to identify why the uri is defective. ### Additional Context This problem appeared in rattler-build, which uses this library. https://github.com/prefix-dev/rattler-build/issues/1595 ### Are you willing to submit a PR to fix this bug? - [x] Yes, I would like to submit a PR. -- 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]
