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

kval extractor #158

Merged
merged 6 commits into from
Jul 16, 2020
Merged

kval extractor #158

merged 6 commits into from
Jul 16, 2020

Conversation

Mzack9999
Copy link
Member

No description provided.

@Mzack9999 Mzack9999 added enhancement Priority: Low This issue can probably be picked up by anyone looking to contribute to the project, as an entry fix Status: Review Needed The issue has a PR attached to it which needs to be reviewed labels Jul 16, 2020
@Mzack9999 Mzack9999 self-assigned this Jul 16, 2020
@Mzack9999 Mzack9999 linked an issue Jul 16, 2020 that may be closed by this pull request
Proposing new type (kval) support for extractors #151
Closed
@Mzack9999 Mzack9999 requested a review from ehsandeep July 16, 2020 09:19
@ehsandeep
Copy link
Member 

$ echo https://hackerone.com | go run main.go -t test.yaml 

                       __     _ 
     ____  __  _______/ /__  (_)
    / __ \/ / / / ___/ / _ \/ / 
   / / / / /_/ / /__/ /  __/ /  
  /_/ /_/\__,_/\___/_/\___/_/   v2                    

    projectdiscovery.io

[WRN] Use with caution. You are responsible for your actions
[WRN] Developers assume no liability and are not responsible for any misuse or damage.
[INF] Found template in current directory: test.yaml
[ERR] Could not parse file 'test.yaml': yaml: line 7: did not find expected key

Getting following error with example template shared here #151

@ehsandeep ehsandeep added the Status: Revision Needed Submitter of PR needs to revise the PR related to the issue. label Jul 16, 2020
@Mzack9999 Mzack9999 mentioned this pull request Jul 16, 2020
Closed
@Mzack9999 Mzack9999 linked an issue Jul 16, 2020 that may be closed by this pull request
Adding extractor support to RAW and DNS request #150
Closed
@ehsandeep
Copy link
Member

Header:- Confirmed, working as expected.

Cookie:- Still checking how this works.

@Mzack9999 Mzack9999 removed Status: Review Needed The issue has a PR attached to it which needs to be reviewed Status: Revision Needed Submitter of PR needs to revise the PR related to the issue. labels Jul 16, 2020
@Mzack9999 Mzack9999 merged commit f96f5f7 into master Jul 16, 2020
@Mzack9999 Mzack9999 deleted the feature-151-kval-extractor branch July 16, 2020 11:19
@ehsandeep ehsandeep added Status: Completed Nothing further to be done with this issue. Awaiting to be closed. and removed Priority: Low This issue can probably be picked up by anyone looking to contribute to the project, as an entry fix enhancement labels Jul 16, 2020
@ehsandeep ehsandeep mentioned this pull request Jul 16, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ehsandeep ehsandeep Awaiting requested review from ehsandeep

Assignees

@Mzack9999 Mzack9999

Labels
Status: Completed Nothing further to be done with this issue. Awaiting to be closed.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Proposing new type (kval) support for extractors Adding extractor support to RAW and DNS request
2 participants
@Mzack9999 @ehsandeep