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

Make view urls API list all urls including multiple urls per SiteNode #8337

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

rolaechea
Copy link

@rolaechea rolaechea commented Feb 14, 2024

Fixes #8335

Currently, the API call JSON/core/view/urls only lists a single URL per site node. However each SiteNode can represent multiple URLs based on the value of query parameters. This PR changes the API code so that all URL that are represented in SiteNodes are returned in JSON/core/view/urls

Copy link

github-actions bot commented Feb 14, 2024

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@rolaechea rolaechea changed the title Make view urls API list all urls including multiple urls per SiteNode https://github.com/zaproxy/zaproxy/issues/8335 Make view urls API list all urls including multiple urls per SiteNode #8335 Feb 14, 2024
@rolaechea rolaechea changed the title Make view urls API list all urls including multiple urls per SiteNode #8335 Make view urls API list all urls including multiple urls per SiteNode #8335 Feb 14, 2024
@rolaechea rolaechea changed the title Make view urls API list all urls including multiple urls per SiteNode #8335 Make view urls API list all urls including multiple urls per SiteNode fixes #8335 Feb 14, 2024
@rolaechea
Copy link
Author

I have read the CLA Document and I hereby sign the CLA.

@rolaechea
Copy link
Author

recheck

zapbot added a commit to zaproxy/cla that referenced this pull request Feb 14, 2024
@thc202 thc202 changed the title Make view urls API list all urls including multiple urls per SiteNode fixes #8335 Make view urls API list all urls including multiple urls per SiteNode Feb 14, 2024
… if applicable.

Signed-off-by: Rafael Olaechea <rafael@nightvision.net>
@rolaechea

This comment was marked as duplicate.

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

Successfully merging this pull request may close these issues.

API to View URLs should return past history
1 participant