ciscoisesdk icon indicating copy to clipboard operation
ciscoisesdk copied to clipboard

ISE CoA API - Wrong Endpoint in SDK

Open maconrad opened this issue 2 years ago • 3 comments

Prerequisites

  • [ ] Have you tested the operation in the API directly? Yes
  • [ ] Do you have the latest SDK version? Yes

Describe the bug API Endoint contains character it should not (>) e_url = ('/admin/API/mnt/CoA/Disconnect>/... instead of e_url = ('/admin/API/mnt/CoA/Disconnect/...

Therefore you get HTTP 400, Request not processed - Bad input

Expected behavior Correct API endpoint.

Screenshots Please provide an screenshot of the successful API call with cuRL, Postman, etc.

image

Environment (please complete the following information):

  • ISE version and patch: 3.1
  • Python version: 3.9
  • SDK version: tested 3.1, 3.1 patch 1
  • OS version: OSX

Additional context Would be nice if you could also name the variables correctly.

image

So e.g. disconnect_type instead of dis_con_nec_tty_pe, psn_name instead of psn_nam_e etc.

Thank you

maconrad avatar Oct 27 '22 08:10 maconrad

Hi @maconrad, The ciscoisesdk v2.0.7 is available. Feel free to test it and close or comment if necessary.

bvargasre avatar Oct 27 '22 20:10 bvargasre

@bvargasre, I think we could remove also the extra {} enclosing the ENDPOINT_IP parameter, or change it in our internal apply_path_params function.

wastorga avatar Oct 31 '22 17:10 wastorga

@wastorga, Fixed it

bvargasre avatar Nov 01 '22 20:11 bvargasre

Closed due to inactivity

bvargasre avatar Nov 16 '22 19:11 bvargasre