Skip to content
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.

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

why http need readline first? #185

Open
JinHeap opened this issue Jun 12, 2024 · 2 comments
Open

why http need readline first? #185

JinHeap opened this issue Jun 12, 2024 · 2 comments

Comments

@JinHeap
Copy link

JinHeap commented Jun 12, 2024

What feature do you want to see added?

I did not use a browser; instead, I used telnet (e.g., telnet 127.0.0.1 9999) to retrieve the status.
I found that I needed to press Enter to display some information. And apart from HTTP, I didn't find any other way to retrieve the status. Actually, using telnet is more convenient than using a browser. I hope to remove readline first.
I tested after removing readline and Chrome browser was working normally.

Upstream changes

e0bede6

Are you interested in contributing this feature?

No response

@centic9
Copy link
Member

centic9 commented Jun 12, 2024

I'd say file-leak-detector simply adheres to the HTTP spec here and thus it wants to read the HTTP GET request properly.

Why don't you use curl for such a thing here instead of telnet?

@JinHeap
Copy link
Author

JinHeap commented Jun 13, 2024

curl is good suggestion. But I use wireshark to check packet from chrome find is so many line in request, does http read first line is must?
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants