Roundup Tracker - Issues

Message8211

Author rouilj
Recipients rouilj
Date 2024-12-08.07:16:20
Message-id <1733642180.38.0.84670733291.issue2551289@roundup.psfhosted.org>
In-reply-to
Fixed in changeset:   8177:2967f37e73e4 as part of refactoring work on 
RestfulInstance::dispatch().

Created a new method: determine_output_format(self, uri) that return a tuple:

  mime_type - None if no matching type found otherwise 'json' or 'xml' for now
  url - modified url to be dispatched if mime type found with '.json' or '.xml' suffix in url
  error - an error object, None if no error found.

If mime_type is None, error is not None and vice versa.

mime_type is a bit of a misnomer. the abbreviated xml and json types are used for
regular rest output because they are short. But for supporting binary_content a
full mime type can be assigned without issue. 

Passed in CI with 97% coverage for the patch. I don't test xml output and
the patch changed that code path.
History
Date User Action Args
2024-12-08 07:16:20rouiljsetmessageid: <1733642180.38.0.84670733291.issue2551289@roundup.psfhosted.org>
2024-12-08 07:16:20rouiljsetrecipients: + rouilj
2024-12-08 07:16:20rouiljlinkissue2551289 messages
2024-12-08 07:16:20rouiljcreate