mattp5657 opened a new issue, #4157:
URL: https://github.com/apache/iggy/issues/4157
### Description
`meilisearch_sink::normalize_host` detects the URL scheme via
`trimmed.starts_with("http://";) || trimmed.starts_with("https://";)`, which is
case-sensitive. A config value like `HTTPS://realhost:9200` matches neither
literal, falls through to the "no scheme" branch, and gets `http://` prepended,
producing `http://HTTPS://realhost:9200`.
### Affected area / component
Connectors
### Proposed solution
Detect the scheme case-insensitively (e.g. `eq_ignore_ascii_case` on the
split-off scheme) instead of a literal `starts_with` check.
### Alternatives considered
_No response_
### Contribution
- [x] I'm willing to submit a pull request to implement this feature
### Good first issue
- [ ] I think this could be a good first issue for a new contributor
--
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]
