httptoolkit / httptoolkit-server

The backend of HTTP Toolkit
https://httptoolkit.com
GNU Affero General Public License v3.0
436 stars 96 forks source link

Interceptor for brave #10

Closed qzaidi closed 4 years ago

qzaidi commented 4 years ago

Is it possible to intercept brave the same way chrome is being intercepted - and launch it in place of chrome

pimterry commented 4 years ago

It's absolutely possible, it's just not implemented quite yet.

The main thing that needs doing is to collect the info required so we can detect whether Brave is installed. If you're a Brave user, any chance you could help with that?

For reference, the change required to detect the new Chromium-based Edge is here: https://github.com/httptoolkit/browser-launcher/commit/d10e68401c9429bb2d543b5efb895ca3bbc423ae?w=1. It looks like https://github.com/vweevers/win-detect-browsers does already supports Brave, so we're good for Windows at least.

To do Mac & Linux, I think we need:

I'll try to take a look at this myself sometime soon, but if you've got any of the above details to hand then that'd definitely help get this implemented sooner :smiley:.

qzaidi commented 4 years ago

Thanks for so quick reply and considering the feature request, surely I can help with the CFBundleVersion on Mac.

        <key>CFBundleVersion</key>
        <string>105.112</string>
        <key>CrProductDirName</key>
        <string>BraveSoftware/Brave-Browser</string>

The binary name on linux is indeed brave

This is the output from running --version on mac, I would guess linux will have similar output /Applications/Brave\ Browser.app/Contents/MacOS/Brave\ Browser --version Brave Browser 80.1.5.112

pimterry commented 4 years ago

Screenshot from 2020-05-07 12-38-31

Now done and live, along with various other browser support improvements :+1:. Your app will update in the background next time it's started, and then pick it up the following run. It's still a bit experimental, so let me know if you hit any issues. Thanks for the suggestion!