kilork / keycloak

Keycloak REST API Client
The Unlicense
62 stars 32 forks source link

Endpoints with `Vec` input doesn't work! #114

Closed omid closed 6 months ago

omid commented 6 months ago

Reqwest doesn't support Vec as input of query method.

We have Vec<String> in two api calls. Here and here.

We should either merge them with "," ourselves or just accept String instead.

kilork commented 6 months ago

The query parameters are actually multiplied in queries. If we have p parameter with value ['v1', 'v2'] in the end it will be p=v1&p=v2, not p=v1,v2. But seems like this really not supported by framework. Thought we can easy fix it. Thanks for reporting!

omid commented 6 months ago

Yes, you are right! Funny that the Keycloak UI (the version I use) sends type=LOGIN,LOGOUT also, but then its backend returns 500 :D But with curl type=LOGIN&type=LOGOUT works

omid commented 6 months ago

@kilork fixed