Skip to content
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

Update to support another implementation with Blazor where messages d… #1

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

R4ML1N
Copy link

@R4ML1N R4ML1N commented Oct 13, 2023

Hi! First of all thanks for your work with this extension. In case it helps, here are some changes I made first to be able to compile the app with the latest version of the montoya-api. Then I made some other changes to support a Blazor APP that I recently ran into. This other app does not follow the same approach of including the _blazor?id as part query string in each request. But instead it included the SignalR header in each request for example. Therefore, I used that to recognize the messages and enable the parsing.

Thanks!

@irsdl
Copy link

irsdl commented Oct 19, 2023

This sounds like a good update, I am going to fork the PR instead of the main project now!
The only issue I can see here is if the project owner does not want to only support the latest version of Burp Suite using the latest version of Montoya. I personally think it is ok to do this given it has some bug fixes but users may need to use the Early edition until the full release is out (I think it will be soon because of Bambdas)

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