-
Notifications
You must be signed in to change notification settings - Fork 2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
member_list is broken #669
Conversation
Now returns lists of (id, type, capacity) tuples where type is a string. Previously, type was a Python class object which could not always be serialized to JSON in the API controller.
Allows a proper error to be generated in the API controller instead of just giving a 500 error.
@amercader This is another bug to consider for 2.0, cheers. |
trans = new_authz.roles_trans() | ||
|
||
def translated_capacity(capacity): | ||
try: | ||
return trans[capacity] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
trans doesn't seem to defined anywhere - do we know what this is about it seems wrong to me - can it just be removed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@tobes I'm not sure what you mean, roles_trans is in new_authz. Whether we actually need translations of the capacity value is another thing, I'm not sure.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sorry - just a case of madness ;p
Now returns lists of (id, type, capacity) tuples where type is a string. Previously, type was a Python class object which could not always be serialized to JSON in the API controller.
Allows a proper error to be generated in the API controller instead of just giving a 500 error.
Doing an API request with a valid group ID causes a server 500 error with the message:
Doing an API request with an invalid group ID causes a server 500 error with the message: