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.

Sign up for GitHub

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

Jump to bottom

Additional global variable support to network protocol #1205

Closed
ehsandeep opened this issue Nov 1, 2021 · 0 comments · Fixed by #1282
Closed

Additional global variable support to network protocol #1205

ehsandeep opened this issue Nov 1, 2021 · 0 comments · Fixed by #1282
Assignees
@Ice3man543
Labels
Priority: Medium This issue may be useful, and needs some attention. Status: Completed Nothing further to be done with this issue. Awaiting to be closed. Type: Enhancement Most issues will probably ask for additions or changes.
Milestone
v2.5.4

Comments

@ehsandeep
Copy link
Member

ehsandeep commented Nov 1, 2021 •
edited
Loading

Please describe your feature request:

Similar to #935 which adds support for {{Host}} and {{Port}} variable for HTTP protocol, we need to extend the same variable to network protocol as well, as it required in cases like below:-

    host:
      - "{{Hostname}}"
      - "{{Host}}:{{Port}}"
      - "{{Host}}:21"
@ehsandeep ehsandeep added Type: Enhancement Most issues will probably ask for additions or changes. Priority: Medium This issue may be useful, and needs some attention. labels Nov 1, 2021
@Ice3man543 Ice3man543 self-assigned this Nov 23, 2021
@Ice3man543 Ice3man543 added the Status: In Progress This issue is being worked on, and has someone assigned. label Nov 23, 2021
@Ice3man543 Ice3man543 linked a pull request Nov 23, 2021 that will close this issue
Added additional variables for network + simplified logic #1282
Merged
4 tasks
@Ice3man543 Ice3man543 mentioned this issue Nov 23, 2021
Merged
4 tasks
@Ice3man543 Ice3man543 linked a pull request Nov 23, 2021 that will close this issue
Added default fields for DNS requests #1284
Merged
4 tasks
@Ice3man543 Ice3man543 removed a link to a pull request Nov 23, 2021
Added additional variables for network + simplified logic #1282
Merged
4 tasks
@Ice3man543 Ice3man543 linked a pull request Nov 23, 2021 that will close this issue
Added additional variables for network + simplified logic #1282
Merged
4 tasks
@Ice3man543 Ice3man543 removed a link to a pull request Nov 23, 2021
Added default fields for DNS requests #1284
Merged
4 tasks
@ehsandeep ehsandeep added Status: Review Needed The issue has a PR attached to it which needs to be reviewed Status: Completed Nothing further to be done with this issue. Awaiting to be closed. and removed Status: In Progress This issue is being worked on, and has someone assigned. Status: Review Needed The issue has a PR attached to it which needs to be reviewed labels Nov 27, 2021
@ehsandeep ehsandeep added this to the v2.5.4 milestone Nov 30, 2021
@ehsandeep ehsandeep mentioned this issue Dec 5, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Ice3man543 Ice3man543

Labels
Priority: Medium This issue may be useful, and needs some attention. Status: Completed Nothing further to be done with this issue. Awaiting to be closed. Type: Enhancement Most issues will probably ask for additions or changes.
Projects
None yet
Milestone
v2.5.4
Development

Successfully merging a pull request may close this issue.

Added additional variables for network + simplified logic projectdiscovery/nuclei
2 participants
@ehsandeep @Ice3man543