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.

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

Adding GitHub organization scanning #48

Closed
wants to merge 24 commits into from

Conversation

samsmithnz
Copy link
Owner

@samsmithnz samsmithnz commented Aug 25, 2022

Dropping the -r argument will scan the owner/organization. Partly addresses #41.

  • First get a list of all repos
  • Iterate through each repo, scanning it
  • How can we handle rate limit exhaustion? Perhaps look at the rate limit and calculate if there is enough calls left to complete the transaction?
  • How can we handle massive organizations? Can we scan an org and if it's more than n abort?
  • Can we add parallel processing. If we get multiple pages of data, can we process each page in parallel?

@samsmithnz samsmithnz marked this pull request as ready for review August 26, 2022 01:22
@samsmithnz
Copy link
Owner Author

This is become complicated - and I'm not sure this is worth it - there are too many situations where an organization has too many repos and this will exhaust rate limits. Officially pausing this feature.

@samsmithnz samsmithnz added the bug Something isn't working label Oct 2, 2022
@samsmithnz
Copy link
Owner Author

.NET 5 bug is fixed in #52

@samsmithnz samsmithnz removed the bug Something isn't working label Oct 2, 2022
@samsmithnz
Copy link
Owner Author

Closing, this was addressed with #132

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant