set elasticsearch[:download_url] to nil

Computing attributes like this, in the attributes file, causes really confusing behavior at runtime. Setting ``elasticsearch[:download_url]`` to nil, and computing at runtime instead. User can override the attribute in a wrapper cookbook and the recipe will actually respect that override.