go-exploit icon indicating copy to clipboard operation
go-exploit copied to clipboard

Add `vhost` and `path`/`rootpath` default flags

Open terrorbyte opened this issue 1 year ago • 2 comments

It's come up a few times in the last few weeks where we have added flags for doing virtualhost settings in protocol or want to allow for changing of the default URI path stem. It seems sensible to probably make these default.

One potential downside is that not everything is HTTP (yet at least), so those flags might be considered nonsense in those cases... but at the same time we have user-agent flags. Probably a discussion for another time but should these be put into target classes that potentially change the default flags?

If I get a few cycles this week I will open a PR to get these in.

terrorbyte avatar Jul 03 '24 15:07 terrorbyte

After some thinking on this we should probably do a few things:

  1. Avoid adding global flags that are protocol assumptive.
  2. Match on the config.Protocol and for a known protocols add flags based on the protocol needs.

terrorbyte avatar Aug 15 '24 17:08 terrorbyte

Match on the config.Protocol and for a known protocols add flags based on the protocol needs.

This strikes me as a particularly good idea. Already we have at least 1 global flag that is always used but is only associated with HTTP (user agent). It seems that would be easy to clean up.

j-baines avatar Aug 16 '24 14:08 j-baines