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

Add common download file extensions #21386

Closed
wants to merge 1 commit into from
Closed

Add common download file extensions #21386

wants to merge 1 commit into from

Conversation

MichaIng
Copy link
Contributor

@MichaIng MichaIng commented Oct 9, 2023

Description:

asc and sig are commonly used for download signatures. md5 and sha are common download hashes.
xz is a common archive type.

A combination of these file extensions can be seen on a lot of websites which offer software downloads, with dedicated checksums/hashes and detached signatures to verify integrity and authenticity for the offered downloads.

Review

asc and sig are commonly used for download signatures.
md5 and sha are common download hashes.
xz is a common archive type.

A combination of these file extensions can be seen on a lot of websites which offer software downloads, with dedicated checksums/hashes and detached signatures to verify integrity and authenticity for the offered downloads.

Signed-off-by: MichaIng <[email protected]>
@MatomoForumNotifications

This pull request has been mentioned on Matomo forums. There might be relevant details there:

https://forum.matomo.org/t/xz-archive-downloads-not-tracked/53528/3

@sgiehl sgiehl added the Needs Review PRs that need a code review label Oct 9, 2023
@github-actions
Copy link
Contributor

This issue is in "needs review" but there has been no activity for 7 days. ping @matomo-org/core-reviewers

@github-actions github-actions bot added the Stale The label used by the Close Stale Issues action label Oct 17, 2023
@bx80
Copy link
Contributor

bx80 commented Oct 17, 2023

buildjs

@bx80 bx80 mentioned this pull request Oct 17, 2023
11 tasks
@bx80
Copy link
Contributor

bx80 commented Oct 17, 2023

Thanks for the pull request @MichaIng, it's much appreciated 😃

Indeed those file extensions are usually downloaded and not shown in the browser so it makes sense to count them as downloads.

I've based a new branch and pull request off your fork in order to add the JavaScript minification, but your original commit will still be attributed. I'll close this PR now.

@bx80 bx80 closed this Oct 17, 2023
@MichaIng MichaIng deleted the patch-1 branch October 17, 2023 21:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs Review PRs that need a code review Stale The label used by the Close Stale Issues action
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants