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

Fixed dead links #15

Merged
merged 4 commits into from
Jun 5, 2021
Merged

Fixed dead links #15

merged 4 commits into from
Jun 5, 2021

Conversation

SemicolonExpected
Copy link
Contributor

Fortify Taxonomy link was dead
Replaced it what I believe to be the equivalent material "Seven Pernicious Kingdoms: A Taxonomy of Software Security Errors" Also from Fortify

There is also the updated: (The Evolution of a Taxonomy: Ten Years of Software Security) https://vulncat.fortify.com/data/HPSR_An_Evolving_Taxonomy_2014.pdf but I'm not sure if you want to include that as well

Fortify Taxonomy link was dead
Replaced it what I believe to be the equivalent material "Seven Pernicious Kingdoms: A Taxonomy of Software Security Errors" Also from Fortify
CMU bomb lab is no longer available without CS:App account. You can only get this with CMU instructor approval
@SemicolonExpected SemicolonExpected changed the title Fixed dead link Fixed dead links May 26, 2021
@ColdHeat ColdHeat merged commit 7ae50bd into osirislab:master Jun 5, 2021
@ColdHeat
Copy link
Contributor

ColdHeat commented Jun 5, 2021

Thanks! Feel like taking on the additional task of vendoring all those PDFs into the repo so that the links dont break in the future?

@SemicolonExpected
Copy link
Contributor Author

I believe the links I fixed are already in the repo, they just didn't have a link to them, unless you mean the taxonomy links

@ColdHeat
Copy link
Contributor

ColdHeat commented Jun 6, 2021

Actually I meant more like copying the PDFs into the repo so that we don't have to worry about links breaking again. We can still cite the source using a link but seems better to just store the PDFs in the repo too.

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.

2 participants