search

DMTF / Redfish-Mockup-Server

A simple Python 3.4 program that can be copied into a folder at the top of any Redfish mockup and can serve Redfish requests on the specified IP/port.
Other
85 stars 38 forks source link

clean_path takes two arguments #46

Closed robisonjoel closed 6 years ago

robisonjoel commented 6 years ago

Calling DELETE would cause a traceback.

tomasg2012 commented 6 years ago

The correct way to fix this would be to make that clean_path call similar to the other calls (using self.shortForm), otherwise it will likely fail when using the -S command.

robisonjoel commented 6 years ago

The correct way to fix this would be to make that clean_path call similar to the other calls (using self.shortForm), otherwise it will likely fail when using the -S command.

Right you are! honestly not sure how I missed that :/

tomasg2012 commented 6 years ago

I missed it first! The function also doesn't detail what that parameter is for, it should be optional.

mraineri commented 6 years ago

Approved 10/25