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

Update version check URL #1120

Merged
merged 1 commit into from
Oct 24, 2015
Merged

Update version check URL #1120

merged 1 commit into from
Oct 24, 2015

Conversation

kylejohnson
Copy link
Member

This commit tells zmupdate.pl to check the version from https://update.zoneminder.com/version.txt

update.zoneminder.com is now hosted on one of my servers, with SSL.

A cronjob fires off every 15 minutes and 1) pulls down the latest version from https://github.com/ZoneMinder/ZoneMinder/blob/master/version. 2) 'pings' DeadMansSnitch

If the cronjob has not succeeded within the last hour, I am alerted (via DeadMansSnitch). This should prevent any silently failing issues.

The reason for this is simple: We can now take a look at the updates.zoneminder.com access log and get a rough estimate on the number of users that we have.

@kylejohnson
Copy link
Member Author

Before this is merged, could someone please do the following (as I seem to have a phantom firewall rule preventing me from testing my own server)

  1. Edit their zmupdate.pl with the change referenced in the above commit
  2. Run zmupdate.pl -c'
  3. Success?

@knight-of-ni
Copy link
Member

looks like success:

2015-10-23 15:06:43.133590  zmupdate    2240    INF Got version: '1.28.108' zmupdate.pl 
2015-10-23 15:06:41.159340  zmupdate    2240    INF Checking for updates    zmupdate.pl

@kylejohnson
Copy link
Member Author

Nice!
12.250.218.126 - - [23/Oct/2015:20:06:44 +0000] "GET /version.txt HTTP/1.1" 200 9 "-" "ZoneMinder Update Agent/1.28.1"

kylejohnson added a commit that referenced this pull request Oct 24, 2015
@kylejohnson kylejohnson merged commit de3e778 into master Oct 24, 2015
@kylejohnson kylejohnson deleted the zmupdate-url branch October 24, 2015 01:02
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