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

Remove MAT #519

Merged
merged 3 commits into from
Aug 3, 2014
Merged

Remove MAT #519

merged 3 commits into from
Aug 3, 2014

Conversation

diracdeltas
Copy link
Contributor

Fix #497
Close #492
Fix #348
Apologize to @gnusosa

…p-depends"

This reverts commit 0e6764a, reversing
changes made to 1d62c31.
This reverts commit 6735ffa, reversing
changes made to c59609d.

Conflicts:
	.travis.yml
	install_files/source-requirements.txt
	securedrop/requirements/source-requirements.txt
	securedrop/source.py
	securedrop/source_templates/lookup.html
	securedrop/static/css/securedrop.css
	securedrop/store.py
	securedrop/tests/test_unit.py
	setup_dev.sh
@garrettr
Copy link
Contributor

garrettr commented Aug 2, 2014

Also need to remove python-poppler from setup_dev.sh and the Debian package. I can do this before merging.

hachoir-core==1.3.3
hachoir-parser==1.3.4
mutagen==1.22
pdfrw==0.1
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

feelsgoodman.jpg

@garrettr
Copy link
Contributor

garrettr commented Aug 2, 2014

Also need to remove test_images/clean.jpg

@garrettr
Copy link
Contributor

garrettr commented Aug 3, 2014

And remove line from the changelog

@diracdeltas
Copy link
Contributor Author

Also need to remove test_images/clean.jpg

This is needed for test_submit_clean_file. Not strictly necessary anymore now that we don't test submitting a dirty file to be scrubbed, but it doesn't hurt to have an extra test for submitting an image file.

@garrettr
Copy link
Contributor

garrettr commented Aug 3, 2014

OMG getting rid of MAT brought back Flask's awesome debug logging! I've been trying to figure that one out for so long. Fixes #348.

@garrettr garrettr merged commit ccdb628 into freedomofpress:develop Aug 3, 2014
@diracdeltas diracdeltas mentioned this pull request Aug 3, 2014
@gnusosa
Copy link
Contributor

gnusosa commented Aug 4, 2014

@diracdeltas @garrettr Sorry, it brought so much pain for you guys :( . I was going to suggest the removal of MAT, and move on to other solution. MAT seems to not be maintained that well anymore. I might be wrong. What solution will you use to scrape Metadata for now?

@gnusosa
Copy link
Contributor

gnusosa commented Aug 4, 2014

Oh no worries I found the conservation. #497
I agree on creating tools outside of SecureDrop for scraping the metadata. Thanks. :)

@garrettr
Copy link
Contributor

garrettr commented Sep 5, 2014

Sorry, it brought so much pain for you guys :(

@gnusosa Thanks for all your work! It's too bad we can't include this in the 0.3 release. If you're interested in doing more work with metadata analysis, I think we're going to start building a tool to help journalists analyze and sanitize metadata after 0.3 is released.

@gnusosa
Copy link
Contributor

gnusosa commented Sep 5, 2014

@garrettr Yeah, I would love to work on that. Let me know if you have issues or any spec on this.

@runasand
Copy link
Contributor

runasand commented Sep 8, 2014

I created #543 for this new tool.

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