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

downloads reporting appears to be broken #554

Closed
yogthos opened this issue Aug 13, 2016 · 6 comments
Closed

downloads reporting appears to be broken #554

yogthos opened this issue Aug 13, 2016 · 6 comments

Comments

@yogthos
Copy link

yogthos commented Aug 13, 2016

All artifacts published since August 5th show 0 downloads.

@tobias
Copy link
Member

tobias commented Aug 13, 2016

It looks like this may be a permission issue - we upgraded nginx, and the stats cronjob may no longer have rights to read the log files. Thanks for the report, I'll take a look.

@tobias
Copy link
Member

tobias commented Aug 13, 2016

Turns out it wasn't a permissions problem, but a slight change in the log format that broke our log parser (an extra space after the date). I'll fix, and regenerate the stats for the missing days.

@yogthos
Copy link
Author

yogthos commented Aug 13, 2016

👍

tobias added a commit that referenced this issue Aug 13, 2016
With the recent nginx update, we changed the access log format slightly:

* 2 spaces after the date instead of 1
* append the sni host to the end of the line

The log parser ignores the sni host, but was strict about the spaces
after the date, causing stat generation to find no downloads. This
relaxes the regex to allow for any number of spaces after the date so we
can re-process logs that were generated with the new format, and allows
us to adjust the format to go back to one space if we decide to do so.

Related to #554
tobias added a commit that referenced this issue Aug 13, 2016
@tobias
Copy link
Member

tobias commented Aug 13, 2016

I've updated the stats and restarted clojars to clear the cached stats. Can you confirm you are seeing downloads for the expected artifacts?

@yogthos
Copy link
Author

yogthos commented Aug 13, 2016

Yup, can confirm everything's in order. Thanks for the quick turn around. ;)

@tobias
Copy link
Member

tobias commented Aug 13, 2016

Great, thanks for the confirmation, and my pleasure!

@tobias tobias closed this as completed Aug 13, 2016
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

No branches or pull requests

2 participants