Add support for download-url function on node/storage

[alperencelik]

The PR aims to add download-url functionality on Proxmox API. Here is the relevant API documentation:

https://pve.proxmox.com/pve-docs/api-viewer/#/nodes/{node}/storage/{storage}/download-url

[alperencelik]

I'm not good at naming so I'm open to any ideas for renaming the function.

[codecov]

Codecov Report

Attention: 3 lines in your changes are missing coverage. Please review.

Comparison is base (fdb95a1) 25.95% compared to head (23c37ac) 25.91%.
Report is 1 commits behind head on main.

Files	Patch %	Lines
nodes.go	0.00%	3 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #110      +/-   ##
==========================================
- Coverage   25.95%   25.91%   -0.05%     
==========================================
  Files          15       15              
  Lines        1753     1756       +3     
==========================================
  Hits          455      455              
- Misses       1278     1281       +3     
  Partials       20       20              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[luthermonson]

I'm not good at naming so I'm open to any ideas for renaming the function.

this api endpoint downloads the url to the storage, i think StorageDownloadURL is probably fine and matches the proxmox api

if motivated there are optional params which i think others could find super useful, might be better to refactor and make a DownloadURLOptions struct which you build the post params from and add the optional params

[alperencelik]

Thanks for the review! I think adding a struct for covering optional parameters is good idea.

[luthermonson]

good work!