Skip to content

Fix import of reports in ZIP files.#2

Open
jangrewe wants to merge 1 commit intopetr22:masterfrom
jangrewe:master
Open

Fix import of reports in ZIP files.#2
jangrewe wants to merge 1 commit intopetr22:masterfrom
jangrewe:master

Conversation

@jangrewe
Copy link
Copy Markdown

@jangrewe jangrewe commented Nov 4, 2018

file_get_contents doesn't seem to play well with ZIP files unless it gets the absolute path to the archive.

@Xunnamius
Copy link
Copy Markdown

My fork has a further fix on top of this, namely: handling gz vs gzip both as gzip (and also fixing the regular expressions, which didn't escape out the periods) @ https://github.com/Xunnamius/DMARC-reports-analyzer

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