Opened 3 months ago

Closed 3 months ago

#1552 closed defect (fixed)

Better check for json format parameters

Reported by: dmisev Owned by: dmisev
Priority: major Milestone: 9.4
Component: conversion Version: development
Keywords: Cc: bphamhuu
Complexity: Medium

Description

In encode/decode there are two different formats for the format parameters (third parameter of these functions): key/value (deprecated) and json.

Currently, we try to parse json; if this fails, we print a warning and try to parse key/value then.

This is not the best approach, as sometimes the format parameters might be json, but fail to parse because the json is invalid. In this case rasdaman should throw an exception instead of printing a warning.

A better test for whether the format parameters are json, is to check if the first non-whitespace character is '{'.

Change History (1)

comment:1 Changed 3 months ago by dmisev

  • Resolution set to fixed
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.