gophertunnel icon indicating copy to clipboard operation
gophertunnel copied to clipboard

NewIncomingConnection + ping (client)

Open fwflunky opened this issue 3 years ago • 5 comments

Hello, I noticed that the NewIncomingConnection is sent along with the ping inside encapsulated packet. Please add the ping packet sending along with the NewIncomingConnection image

fwflunky avatar Aug 31 '21 14:08 fwflunky

Why?

Sandertv avatar Aug 31 '21 14:08 Sandertv

Why?

The server can check for a ping packet with a NewIncomingConnection in the same encapsulated packet, then filter the bot. I think you need to make a bypass (sorry for my bad english)

fwflunky avatar Aug 31 '21 15:08 fwflunky

Nobody would ever use this to check against gophertunnel because nothing prevents the mcpe client from doing this. I suppose it can be changed but it is in no way a concern. People checking against this would just be shooting their own foot

Sandertv avatar Aug 31 '21 16:08 Sandertv

I suppose it can be changed

No need to close this. If anyone feels like changing it it'll be changed.

Sandertv avatar Sep 01 '21 10:09 Sandertv

Sorry for the snappy answers earlier, that wasn't very cool.

Sandertv avatar Sep 01 '21 10:09 Sandertv