Skip to content

dcache rest api: do not filter out fields with default and null values#7843

Merged
mksahakyan merged 1 commit into11.0from
14479/11.0
Jul 14, 2025
Merged

dcache rest api: do not filter out fields with default and null values#7843
mksahakyan merged 1 commit into11.0from
14479/11.0

Conversation

@DmitryLitvintsev
Copy link
Member

when building json reply object. Enable get/set of OUTPUT quota

Motivation:

dCache rest api response filters out fields that have null or default (for that type) values. As the result, for instance, integer filed with 0 value is skipped (so is any filed with null value). This is not desirable

Modification:

Change custom Json ObjectMapper to retain fields with null and default values

Result:

Request results returned by dCache rest api contain fields having null or default values

Ticket: #7836
Patch: https://rb.dcache.org/r/14479/
Target: trunk
Request: 11.0, 10.2, 10.1, 10.0, 9.2

Acked-by: Tigran

Require-book: no
Require-notes: yes

when building json reply object. Enable get/set of OUTPUT quota

Motivation:
-----------

dCache rest api response filters out fields that have null
or default (for that type) values. As the result, for instance,
integer filed with 0 value is skipped (so is any filed with null
value). This is not desirable

Modification:
-------------

Change custom Json ObjectMapper to retain fields with null and default
values

Result:
-------

Request results returned by dCache rest api contain fields having
null or default values

Ticket: #7836
Patch: https://rb.dcache.org/r/14479/
Target: trunk
Request: 11.0, 10.2, 10.1, 10.0, 9.2

Acked-by: Tigran

Require-book: no
Require-notes: yes
@mksahakyan
Copy link
Contributor

retest this please

@mksahakyan mksahakyan merged commit e95887c into 11.0 Jul 14, 2025
1 of 3 checks passed
@mksahakyan mksahakyan deleted the 14479/11.0 branch July 14, 2025 12:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants