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

add docs for batch endpoint #169

Conversation

joephylum
Copy link
Contributor

@joephylum joephylum commented Jun 25, 2024

Adds docs for new /batch threat feed end point. Also adds docs about top level key additions to single package end point.

Note that I will update the example in the docs with one that actually returns slightly more interesting results after this gets deployed and I can run it against prod instead of staging.

@joephylum joephylum marked this pull request as ready for review June 25, 2024 23:36
@joephylum joephylum requested a review from a team as a code owner June 25, 2024 23:36
Copy link
Contributor

@janasheehan janasheehan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

perhaps i am the wrong person to review this... but i tried to be a user and nothing worked for me. otherwise, it looks good (not fond of adrmdr, but that's me)

@joephylum joephylum requested a review from louislang June 26, 2024 23:42
@joephylum
Copy link
Contributor Author

Thanks @janasheehan! Your name was randomly picked to review when I opened the PR, so please don't feel obligated to review it too closely! I'll request some others with more specific knowledge of the endpoint. Thanks again!

…dpoint-and-update-single-package-endpoint-docs
Copy link
Contributor

@ejortega ejortega left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

"riskScores": {
"author": 1.0,
"engineering": 1.0,
"license": 1.0,
"malicious_code": 0.01,
"total": 0.05,
"vulnerability": 1.0
}
},
"version": "0.1.0"
Copy link
Contributor

@ejortega ejortega Jun 27, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe a nitpick, but maybe the example response can match same order specified?

  • name: The name of the package.
  • version: The version of the package.
  • ecosystem: The ecosystem to which the package was published.
  • code: The HTTP status code.
  • malware: A boolean indicating if the package has been triaged as malware.
  • riskScores: An object containing risk scores for each of the 5 Phylum risk domains associated with the pa

…dpoint-and-update-single-package-endpoint-docs
@furi0us333
Copy link
Contributor

Please hold off on merging/publishing this for now.

@joephylum
Copy link
Contributor Author

Please hold off on merging/publishing this for now.

Sounds good! Will convert to draft just to make sure it doesn't get merged accidentally.

@joephylum joephylum marked this pull request as draft July 8, 2024 20:55
@furi0us333
Copy link
Contributor

I am going to close this PR as we will not be including this text in the published docs.

@furi0us333 furi0us333 closed this Jul 15, 2024
@furi0us333 furi0us333 deleted the joe/168-create-docs-for-threat-feed-batch-endpoint-and-update-single-package-endpoint-docs branch July 15, 2024 20:51
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.

Create docs for threat-feed /batch endpoint and update single package endpoint docs
4 participants