openbmc / phosphor-rest-server

REST server that transposes dbus interfaces to REST
Apache License 2.0
4 stars 11 forks source link

logout post requires empty data #13

Open bradbishop opened 8 years ago

bradbishop commented 8 years ago

At the moment when logging out you have to have a request body with { "data": [] } which is goofy. Probably the best thing to do here is have a seperate logout route handler that doesn't install the jsonapirequestformat plugin.

williamli80 commented 8 years ago

PR: https://github.com/openbmc/phosphor-rest-server/pull/22 Hi, please review. New way of logout: curl -c cjar -b cjar -k https://BMC/logout And I retain the json format logout. Is it ok?

williamspatrick commented 8 years ago

@williamli80 - I am prioritizing this for the August sprint. It seems like there were issues with the previous pull request. If you have something close to working, that is fine to continue on this, but I have a few other stories assigned to you for July.

williamli80 commented 8 years ago

Got it.

geissonator commented 7 years ago

Any updates here @williamli80 ?

geissonator commented 7 years ago

Any updates here @williamli80 ?

geissonator commented 7 years ago

Going to queue this one up for one of our meetings

geissonator commented 7 years ago

Low priority so just moving to backlog for now.