dell / iDRAC-Redfish-Scripting

Python and PowerShell scripting for Dell EMC PowerEdge iDRAC REST API with DMTF Redfish
GNU General Public License v2.0
609 stars 279 forks source link

Allow download of local Support assist without the need of a GUI #227

Closed shade34321 closed 2 years ago

shade34321 commented 2 years ago

Instead of using the webbrowser to download the support assist, we can now directly hit the endpoint and write the file locally.

Also moved the accept to earlier since before it was erroring out for us. Also allow checking if we already accepted it and only accepting it if we haven't accepted the EULA.

texroemer commented 2 years ago

@shade34321

Thanks for the suggestions and code changes!

-Tex

shade34321 commented 2 years ago

Thank you! If you don't mind, can you mark this as hacktoberfest-accepted :)

https://hacktoberfest.com/participation/#maintainers

texroemer commented 2 years ago

Done. I also had to make a few changes to the script, some error conditions were not handled correctly especially when trying to pass in arguments export and accept at the same time. Also updated the script examples and removed download argument.

shade34321 commented 2 years ago

Thanks, sorry about that. I would have fixed them had I known.

texroemer commented 2 years ago

No worries :)