Skip to content

http: look for trailing 'type=' in ftp:// without strstr #19065

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Closed
bagder wants to merge 2 commits into from
Closed

http: look for trailing 'type=' in ftp:// without strstr #19065

bagder wants to merge 2 commits into from

Conversation

@bagder
Copy link
Member

@bagder bagder commented Oct 14, 2025 •
edited
Loading

  • it could find a wrong string
  • this is faster

@bagder bagder added the FTP label Oct 14, 2025
@github-actions github-actions bot added the HTTP label Oct 14, 2025
@bagder
http: look for trailing 'type=' in ftp:// without strstr
407eb0f 
- it could find a wrong string
- this is faster
@bagder bagder marked this pull request as ready for review October 14, 2025 16:10
@bagder bagder closed this in 9441127 Oct 15, 2025
@bagder bagder deleted the bagder/ftp-type branch October 15, 2025 06:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jay jay jay approved these changes

Assignees

No one assigned

Labels

FTP HTTP

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@bagder @jay