dell / iDRAC-Redfish-Scripting

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

URI incorrect for older iDRACs #219

Closed blazarmilkywayflybee closed 2 years ago

blazarmilkywayflybee commented 2 years ago

iDRAC 7 and 8 do not support SessionService in URI. Returns 400 Bad Request.

Tested on: iDRAC7 - 2.65.65.65 iDRAC8 - 2.82.82.82 iDRAC9 - 5.10.00.00

blazarmilkywayflybee commented 2 years ago

I've just updated this to mirror that of the Python module with the addition of get_redfish_version function. Also, formatting.

texroemer commented 2 years ago

Hi @blazarmilkywayflybee

Thanks for the catch and adding the code. After merge i noticed the function was missing the check for Powershell version which i updated.

Thanks Tex

blazarmilkywayflybee commented 2 years ago

Thanks @texroemer !