Age | Commit message (Collapse) | Author | |
---|---|---|---|
2015-04-16 | Mark unused variables as unused. | Steve Kemp | |
We do that with the standard "_"-prefix. | |||
2015-04-16 | Mark variables as unused. | Steve Kemp | |
We've now prefixed our transient/unused variables with "_" to stop any future errors about variables not being used. | |||
2015-04-16 | Correctly initialize our state. | Steve Kemp | |
We'd failed to set the default value of our loaded-marker to false, although in-practice this wasn't a problem. | |||
2015-04-16 | Ensure we load our base-class. | Steve Kemp | |
2015-04-16 | Fixed to be valid. | Steve Kemp | |
Due to some sloppy edits this module was not correct. | |||
2015-04-16 | Fixed syntax error. | Steve Kemp | |
2015-04-16 | Correctly use the result of the resolution in the alert. | Steve Kemp | |
There was a bug in that we used the same local-variable "result" for two purposes, trashing the intended use-case. | |||
2015-04-16 | Resolve hostnames for both A + AAAA. | Steve Kemp | |
The Bytemark-specific notifier, using mauve, appends some text to the bottom of each alert: http://example.com/ resolves to 1.2.3.4 which is INSIDE|OUTSIDE bytemark This text was previously limited to the IPv4 address, but now is repeated for each family which resolves successfully. This closes #10568. This fixes #10568. | |||
2015-03-10 | Log the failure-reason on the temporary-fails. | Steve Kemp | |
2015-03-09 | Show error-message clearly on connection-failure. | Steve Kemp | |
This was failing because '$ERROR_INFO' is only available if you require 'English' | |||
2015-03-09 | Updated to test for more whitespace issues. | Steve Kemp | |
2015-03-09 | Removed trailing whitespace from the code | Steve Kemp | |
2015-03-09 | Remove spaces inside empty hashes | Steve Kemp | |
2015-03-09 | Remove spaces inside blocks. | Steve Kemp | |
2015-03-09 | More minor space fixups | Steve Kemp | |
2015-03-09 | Whitespace fixups. | Steve Kemp | |
These were all identified and suggested by rubocop. | |||
2015-03-09 | Use def with parentheses when there are parameters. | Steve Kemp | |
2015-03-09 | Removed spaces inside parenthesis. | Steve Kemp | |
2015-03-09 | Prefer single-quotes when you don't need interpolation. | Steve Kemp | |
So "foo" is less good than 'foo'. | |||
2015-03-09 | Prefer single-quoted strings inside interpolations. | Steve Kemp | |
2015-03-09 | Do not terminate expressions with ";". | Steve Kemp | |
Ruby is not Perl, much as I sometimes wish it were. | |||
2015-03-09 | Avoid redudent returns. | Steve Kemp | |
The last expression of a method is the return value. So: def foo; false ; end Is the same as: def foo; return false; end | |||
2015-03-09 | Avoid bogus casing of method-name. | Steve Kemp | |
So "getURL" becomes "get_url_contents" | |||
2015-03-09 | Don't use parenthesis aroudn conditions in an if. | Steve Kemp | |
2015-03-09 | Do not use parentheses for method calls with no arguments. | Steve Kemp | |
This is neater. Flagged by rubocop | |||
2015-03-09 | Avoid "Array.new" and "Hash.new" | Steve Kemp | |
Instead use {} + (). | |||
2015-03-09 | Removed bogus period. | Steve Kemp | |
2015-03-09 | Omit the parentheses in defs when the method doesn't accept any arguments. | Steve Kemp | |
This is neater. | |||
2015-03-09 | Minor indentation fixup. | Steve Kemp | |
2015-03-09 | Don't use "::" for method-calls. | Steve Kemp | |
Instead prefer ".". Flagged by rubocop | |||
2015-03-09 | Avoid arguments we're not using. | Steve Kemp | |
Having methods take arguments which are ignored is a misleading thing, prefix with "_" to make that explicit, or remove. | |||
2015-03-09 | Removed dead code. | Steve Kemp | |
After raising an exception the following code could not be reached. | |||
2015-03-09 | Avoid redundent ".to_s" methods. | Steve Kemp | |
These are not required if the argument is string already, or has a _to_s method which will be automatically invoked by magic. | |||
2015-03-09 | Don't rescue the 'Exception' | Steve Kemp | |
Instead rescue a 'StandardError' which is slightly more specific. (Rescuing more specific exceptions is good.) | |||
2015-03-09 | Avoid useless wrapping of return-value. | Steve Kemp | |
This: def foo line end Is the same as this: def foo return( line ) end | |||
2015-03-09 | File.exists? is deprecated. | Steve Kemp | |
We prefer "File.exist?". Flagged by rubocop. | |||
2015-03-09 | Use long-names | Steve Kemp | |
Instead of "$0" use $PROGRAM_NAME, instead of $! use "$LOAD_PATH". This is more explicit and less-magic. Flagged by rubocop | |||
2015-03-04 | Allow SSL-check to be disabled, via "no_ssl_check" | Steve Kemp | |
2015-03-04 | New release - reporting SSL errors correctly. | Steve Kemp | |
2015-03-04 | Work correctly if SSL library is too old for SNI. | Steve Kemp | |
2015-03-04 | Reinstate the default logger as - file. | Steve Kemp | |
2015-03-04 | Updated comments and code-order. No functional change. | Steve Kemp | |
2015-03-04 | Don't use a logger. | Steve Kemp | |
Instead we run quietly if nothing was output, and run verbosely if we see "--verbose". | |||
2015-03-04 | New releaserelease-0.24.2 | Steve Kemp | |
2015-03-04 | Allow tests to be qualified IPv4/IPv6-only. | Steve Kemp | |
Via "ipv6_only" and "IPv4_only" | |||
2015-03-04 | Explictly convert the class to a string. | Steve Kemp | |
This is required under Ruby 1.8, as I discovered when deploying to offsite3. | |||
2015-03-02 | Explicitly abort on unknown test-types | Steve Kemp | |
2015-03-02 | Allow IPv4 and IPv6 tests to be carried out. | Steve Kemp | |
If a target is a hostname we'll explicitly resolve it for both IPv4 and IPv6. | |||
2015-03-02 | Added named-test. | Steve Kemp | |
THis tests that a server is listening on :53. | |||
2015-03-02 | Test for IPv4 and IPv6 addresses explicitly. | Steve Kemp | |
If we're given an IPv4 or IPv6 address then use it, if not then attempt to resolve the name that we've been given to one/other/both of these types and test in turn. |