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

ZoomEye - Adding maximum number of items per page check in API #333

Closed
lu4nx opened this issue Nov 9, 2023 · 0 comments · Fixed by #335
Closed

ZoomEye - Adding maximum number of items per page check in API #333

lu4nx opened this issue Nov 9, 2023 · 0 comments · Fixed by #335
Assignees
@dogancanbakir
Labels
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
v1.0.8

Comments

@lu4nx
Copy link
Contributor

lu4nx commented Nov 9, 2023

ZoomEye API returns maxium 20 results per page[1]. If the data is less than 20, uncover will request the next page again while ZoomEye returns an 403 error:

$ ZOOMEYE_API_KEY=<your key> ./uncover -ze shellcodes.org -v

  __  ______  _________ _   _____  _____
 / / / / __ \/ ___/ __ \ | / / _ \/ ___/
/ /_/ / / / / /__/ /_/ / |/ /  __/ /    
\__,_/_/ /_/\___/\____/|___/\___/_/

		projectdiscovery.io

[INF] Current uncover version v1.0.7 (latest)
[zoomeye] 104.243.16.124:443
[zoomeye] 104.225.156.51:443
[zoomeye] 104.225.156.32:443
[zoomeye] unexpected status code 403 received from https://api.zoomeye.org/host/search?query=shellcodes.org&page=2

Refer:

  1. https://github.com/knownsec/ZoomEye-python/blob/master/zoomeye/sdk.py#L192
@lu4nx lu4nx added the Type: Enhancement Most issues will probably ask for additions or changes. label Nov 9, 2023
@lu4nx lu4nx mentioned this issue Nov 9, 2023
Closed
@dogancanbakir dogancanbakir self-assigned this Nov 10, 2023
@dogancanbakir dogancanbakir mentioned this issue Nov 10, 2023
Merged
@dogancanbakir dogancanbakir linked a pull request Nov 10, 2023 that will close this issue
limit zoomeye reqs per total #335
Merged
@ehsandeep ehsandeep added the Status: Completed Nothing further to be done with this issue. Awaiting to be closed. label Nov 10, 2023
@ehsandeep ehsandeep added this to the v1.0.8 milestone May 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dogancanbakir dogancanbakir

Labels
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
v1.0.8
Development

Successfully merging a pull request may close this issue.

limit zoomeye reqs per total projectdiscovery/uncover
3 participants
@lu4nx @ehsandeep @dogancanbakir