Skip to content

wip: handle chunked encoding in /Discover POST body #116

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

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

vergenzt
Copy link

Should resolve #115

@ananyatimalsina
Copy link

Has this been tested with any recent version of IOS/macOS? #109 and #119 mention that Apple seems to have changed the airdrop protocol and thus only versions up to macOS Ventura and iOS 15 are supported, as stated by this project's README. My own tests (conducted with IOS 18) result in having to change

chunk_size = int(self.rfile.readline().rstrip(b"\r\n"),)
to
chunk_size = int(self.rfile.readline().rstrip(b"\r\n"), 16)

and finally conclude with a "No route to host" error on this line: self.rfile.read(), though I will have to investigate further whether this is caused by owl (However it does seem to be fine as it is showing peer add and remove logs) or perhaps even nexmon's monitor mode patch.

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

Successfully merging this pull request may close these issues.

NoneType error macOS 14.4
2 participants