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

Get a diff of the dependencies between commits should explain why it doesn't work for public forks of repositories without dependency graphs enabled #36252

Open
1 task done
jsoref opened this issue Feb 11, 2025 · 2 comments
Labels
content This issue or pull request belongs to the Docs Content team needs SME This proposal needs review from a subject matter expert waiting for review Issue/PR is waiting for a writer's review

Comments

@jsoref
Copy link
Contributor

jsoref commented Feb 11, 2025

Code of Conduct

What article on docs.github.com is affected?

https://docs.github.com/en/rest/dependency-graph/dependency-review?apiVersion=2022-11-28

What part(s) of the article would you like to see updated?

https://docs.github.com/en/rest/dependency-graph/dependency-review?apiVersion=2022-11-28#get-a-diff-of-the-dependencies-between-commits--status-codes

Status code Description
403 Response if GitHub Advanced Security is not enabled for this repository

Here's a public repository using the api:

$ gh api /repos/check-spelling/check-spelling/dependency-graph/compare/HEAD...HEAD~
[]

(I presume that's a 200)

Here's a public fork of the above repository:

$ gh api /repos/check-spelling-sandbox/check-spelling/dependency-graph/compare/HEAD...HEAD~
{
  "message": "Forbidden",
  "documentation_url": "https://docs.github.com/rest",
  "status": "403"
}
gh: Forbidden (HTTP 403)

The docs say that this only happens when Advanced Security is not enabled. Conceptually, Advanced Security is only required for private repositories.

Additional information

Related to:

As with that item, the help needs to suggest enabling the feature...

@jsoref jsoref added the content This issue or pull request belongs to the Docs Content team label Feb 11, 2025
@github-actions github-actions bot added the triage Do not begin working on this issue until triaged by the team label Feb 11, 2025
@subatoi subatoi added needs SME This proposal needs review from a subject matter expert and removed triage Do not begin working on this issue until triaged by the team labels Feb 11, 2025
Copy link
Contributor

Thanks for opening an issue! We've triaged this issue for technical review by a subject matter expert 👀

@subatoi
Copy link
Contributor

subatoi commented Feb 11, 2025

Many thanks @jsoref—I'll get this triaged and reviewed appropriately

@subatoi subatoi added the waiting for review Issue/PR is waiting for a writer's review label Feb 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content This issue or pull request belongs to the Docs Content team needs SME This proposal needs review from a subject matter expert waiting for review Issue/PR is waiting for a writer's review
Projects
None yet
Development

No branches or pull requests

3 participants
@jsoref @subatoi and others