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

Added endpoint extraction from JS file module #320

Closed
wants to merge 5 commits into from
Closed

Added endpoint extraction from JS file module #320

wants to merge 5 commits into from

Conversation

aman566
Copy link
Collaborator

@aman566 aman566 commented Jul 19, 2020 •
edited
Loading

This module extracts all the urls and endpoints in a JS file Using a regex.

Checklist

  • I have followed the Contributor Guidelines.
  • The code has been thoroughly tested in my local development environment with flake8 and pylint.
  • The code is both Python 2 and Python 3 compatible.
  • The code follows the PEP8 styling guidelines with 4 spaces indentation.
  • This Pull Request relates to only one issue or only one feature
  • I have referenced the coresspoinding issue number in my commit message
  • I have added the relevant documentation.
  • My branch is up-to-date with the Upstream master branch.

Your development environment

  • OS: Linux
  • OS Version: Ubuntu 18.04
  • Python Version: 3.6.9

@aman566 aman566 requested a review from securestep9 July 19, 2020 07:53
@aman566 aman566 added the new module pull request with new module(s) label Jul 20, 2020
@Ali-Razmjoo
Copy link
Collaborator

this can't be supported anymore. modules are yaml #440

@Ali-Razmjoo Ali-Razmjoo closed this Sep 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@securestep9 securestep9 Awaiting requested review from securestep9 securestep9 is a code owner

Assignees
No one assigned
Labels
new module pull request with new module(s)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@aman566 @Ali-Razmjoo