Skip to content

Conversation

kaifcodec
Copy link
Contributor

Replaced deprecated

const connectFn = (
    Module.findExportByName('libc.so', 'connect') ?? // Android
    Module.findExportByName('libc.so.6', 'connect') ?? // Linux
    Module.findExportByName('libsystem_kernel.dylib', 'connect') // iOS
);

with

let connectFn = null;
try {
    connectFn =
        Process.getModuleByName('libc.so').findExportByName('connect') ?? //Android
        Process.getModuleByName('libc.so.6').findExportByName('connect') ?? // Linux
        Process.getModuleByName('libsystem_kernel.dylib').findExportByName('connect'); //IOS
} catch (e) {
    console.error("Failed to find 'connect' export:", e);
}

as per Frida v17 changes.
Tested with both Frida v16 and v17, works as expected.

@CLAassistant
Copy link

CLAassistant commented Jun 11, 2025

CLA assistant check
All committers have signed the CLA.

@pimterry pimterry merged commit 362ea92 into httptoolkit:main Jun 20, 2025
4 checks passed
@pimterry
Copy link
Member

Thanks @kaifcodec! I had to tweak this and extend it to cover a few more cases, but it seems it's now all working nicely for both Frida 16 & 17 👍

In case you're not aware, HTTP Toolkit Pro is totally free for all contributors. I've just set up an account for the email from your github profile - just click 'Get Pro' then 'Log into existing account' and enter your email to get started.

@kaifcodec
Copy link
Contributor Author

Thanks a lot, Tim! I really appreciate you reviewing and refining the patch — and the Pro access was an awesome surprise. Happy to contribute, and looking forward to helping out more in the future!

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.

3 participants