Infinite horizontal scrolling - Burp v2020.1

[Hannah-PortSwigger]

We’ve just released a new version of Burp that has some big changes to the HTTP message editor.

Notably, a scroll pane is now implemented inside the message editor.

Unfortunately, this has caused an infinite scrolling issue with your extension. This was reported to us by a user of your BApp, who provided a video of the issue (https://monosnap.com/file/Gqw1tXkXKupRRSAsBSEN3rcjbAYyDw).

If possible, could you remove the scroll pane instance you have implemented, and create a pull request against the PortSwigger branch of your repository so that we can review the changes and update the BApp?

[mattymcfatty]

ditto

[mattymcfatty]

hey @HannahLaw-Portswigger is this change documented anywhere? I tried poking around to change the code myself but wasn't sure what to replace the references to JScrollPane with. I couldn't find references to scroll functions in burp extender online anywhere. Could you help point me in the right direction?

[Hannah-PortSwigger]

Removing the instance of JScrollPane in line 714 seems to fix the infinite scrolling issue. After that, it would be a matter of sorting out the UI again so that it looks nice (I would guess restricting the initial size of the message editor).

I don't have any documentation to provide you with, unfortunately.

mod0BurpUploadScanner/UploadScanner.py

Line 714 in a7ba731

splitpane.setRightComponent(JScrollPane(sc))

[Hannah-PortSwigger]

This issue should be resolved in the next release of Burp.

[putsi]

This issue seems to still be present on UploadScanner that is loaded from BApp store and installed in Burp v2020.2.1.
I tried the fix provided by @HannahLaw-Portswigger but it completely removed the whole right pane.

[Hannah-PortSwigger]

This issue was fixed shortly after the release of 2020.2.1, so it was unable to be included in that release. We have not yet released 2020.3 (but it shouldn't be too far off).

[putsi]

Ok, do you know if there is a hotfix that I can apply? I tried removing the line 714 but it completely removes the panel.

[Hannah-PortSwigger]

You could replace the scroll pane instance with a panel of some type. However, we have fixed the root cause of the issue, which will be in the next release.

[bluecanarybe]

@HannahLaw-Portswigger

I've been waiting for this, thanks!

[Hannah-PortSwigger]

We're pleased to announce the release of Burp v2020.4

[putsi]

Can confirm that the bug is now fixed on Burp v2020.4.