-
Notifications
You must be signed in to change notification settings - Fork 660
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
Migrate Favicon DB Content #684
Comments
Hello Sir, I am Himanshu. I am interested to work on this. Can you please assign me this. |
It would probably been a good move to implement this as a data file with a template/include. That way maintenance will be easier. For an example check the api-tools yaml and html in the repo. |
Sure Sir, I will do it as fast as possible.
…On Sat, Jan 14, 2023 at 12:21 AM Rick M ***@***.***> wrote:
It would probably been a good move to implement this as a data file with a
template/include. That way maintenance will be easier.
For an example check the api-tools yaml and html in the repo.
—
Reply to this email directly, view it on GitHub
<#684 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AXSTOVGQ73FB3UXSU5J4MZTWSGPZXANCNFSM6AAAAAAS5UOAQI>
.
You are receiving this because you were assigned.Message ID:
***@***.***>
|
Sir, I am not getting what's actually the issue here. Can you please explain that and will you please tell me how to run the project? |
That was covered in earlier comments. Migrate the content from the old wiki, into a data file and applicable display components (similar to api-tools already in this repo). Info on local testing can be found here: https://owasp.org/migration/ (toward the bottom of the page). |
@HimanshHK do you still intend to tackle this? |
Hey @kingthorin | Can you assign me this? |
As read earlier in the comments, referencing from - name: "penguin"
id: "6399cc480d494bf1fcd7d16c42b1c11b" Is this good to go? |
Maybe include an Also you'll have to create a new template like: https://github.com/OWASP/www-community/blob/08bbc8291cf9b4b40d0df7ad1efd74693b598ec8/_includes/api-tools.html |
There are some more entries and sources here that you could add as well: https://vulners.com/openvas/OPENVAS:136141256231020108 |
Understood, will update you soon with the PR :) |
https://wiki.owasp.org/index.php/OWASP_favicon_database is referenced by https://tryhackme.com/room/contentdiscovery
we should migrate it to the new site.
The text was updated successfully, but these errors were encountered: