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

[WIP][Enhancement] Add Parsing for sitemap.xml #1000

Closed
wants to merge 1 commit into from
Closed

[WIP][Enhancement] Add Parsing for sitemap.xml #1000

wants to merge 1 commit into from

Conversation

n3rada
Copy link

@n3rada n3rada commented Oct 4, 2023

Hi,

Building upon the discussion in the issue I opened earlier, I believe adding the capability to parse the sitemap.xml file could significantly benefit the tool, especially when the file is present on a target website.

This is my first foray into Rust development, so I've provided this initial code as a starting point.

I have note created appropriated methods inside the test module either.

Hope it could help.

@n3rada n3rada changed the title feat: Adding sitemap.xml parsing [Enhancement] Add Parsing for sitemap.xml Oct 4, 2023
@epi052
Copy link
Owner

epi052 commented Oct 7, 2023

hey there, thanks for putting this together!

I'm going to leave this open for now and mark it WIP (work in progress), but it will take some more work to get this to a place where it's ready to merge.

I'm going to add more comments / info into the original issue request.

@epi052 epi052 changed the title [Enhancement] Add Parsing for sitemap.xml [WIP][Enhancement] Add Parsing for sitemap.xml Oct 7, 2023
Copy link

stale bot commented Dec 15, 2023

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Dec 15, 2023
@stale stale bot closed this Mar 13, 2024
@n3rada
Copy link
Author

n3rada commented Mar 13, 2024

@epi052, what do you think about this issue? Would you have time to work into it, or should we close it as the bot says? 🥺

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

Successfully merging this pull request may close these issues.

2 participants