sipcapture / homer-app

HOMER 7.x Front-End and API Server
http://sipcapture.io
GNU Affero General Public License v3.0
207 stars 85 forks source link

Fix swagger defs #331

Closed ghmj2417 closed 4 years ago

ghmj2417 commented 4 years ago

I tried to do what I could with updating swagger to the correct URIs and HTTP methods. I was not able to test all of the calls via swagger. I think the swagger stuff needs significant review beyond this PR.

Some of the swagger models that reference certain structs are very confusing, also the responses in the swagger definition. Those are very confusing or lacking. I attempted to look at schema definitions and add or update where I could but those were confusing as well.

It would definitely be helpful to add additional models or parameters so users like myself know what data to pass. Same goes for the responses.

adubovikov commented 4 years ago

Great job! thank you!