Skip to content

Added feature request/response title #17

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

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

Conversation

sk3l10x1ng
Copy link

@sk3l10x1ng sk3l10x1ng commented Jul 13, 2025

added title helps to distinguish between the Request/ Response

image

@sk3l10x1ng sk3l10x1ng changed the title add request/response title Added feature request/response title Jul 13, 2025
@tobiashort
Copy link
Contributor

Hi @sk3l10x1ng

Thanks for your contribution.

I understand what you're trying to achieve, but I don’t think I can accept your changes in their current form. Your solution is quite opinionated. Some users neither need nor want these titles. For example, in our case, we use a different mechanism that automatically adds the correct titles when pasting into reports.

Others may prefer different title formats, such as “HTTP request:” / “HTTP response:”, or underlines using ====, or no ASCII underlines at all. I hope you see what I mean. This kind of formatting should be configurable, both in terms of whether titles are added and how they are styled.

There is already a configuration class in place that could give you an idea of how to make this more flexible. Keep in mind that this will likely change again once the new version of Burp Suite is released, which will introduce a built-in configuration API.

If you're interested, feel free to revise your solution. Otherwise, I might revisit this myself sometime in the future.

@sk3l10x1ng
Copy link
Author

@tobiashort an option to choose the type of format “HTTP request:” / “HTTP response:”, or underlines using ==== . This can be done

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.

2 participants