View Issue Details

IDProjectCategoryView StatusLast Update
0007397GNUnetnamestore servicepublic2022-10-31 03:22
Reporterschanzen Assigned ToChristian Grothoff  
PrioritynormalSeveritytweakReproducibilityhave not tried
Status closedResolutionfixed 
Target Version0.18.0Fixed in Version0.18.0 
Summary0007397: Improve expiration time in namestore record JSON
DescriptionWe should use the same
data type that GNUNET_JSON_ creates for absolute/releative time, instead of
having an ill-defined human-readable time here.

Furthermore, I'm not sure it is ideal to have the "relative_expiration" that
separated. I propose:
expiration {
  is_relative : true;
  relative_expiration : RelativeTime;
}

or

expiration {
  is_relative : false;
  absolute_expiration : AbsoluteTime;
}

which is the format that is also easiest to parse in
TypeScript (using 'is_relative' as the type discriminator).
TagsNo tags attached.

Activities

schanzen

2022-10-21 15:00

administrator   ~0019263

The proposed format it really annoying to parse using our parser as it does not allow for optional fields.
So there is a condition now (is_relative) that determines the format, but we have to parse first to know the format. Which is really strange.
I suggest we keep 1 number field that is named "expiration_time" and its contents are determined by is_relative. Just like it is now.

schanzen

2022-10-23 15:38

administrator   ~0019277

Nvm I was thinking of how this needs to be parsed in Go
9c7255313..6d56e7dc6

Issue History

Date Modified Username Field Change
2022-10-18 13:46 schanzen New Issue
2022-10-19 09:27 schanzen Assigned To => schanzen
2022-10-19 09:27 schanzen Status new => assigned
2022-10-21 15:00 schanzen Note Added: 0019263
2022-10-21 15:26 schanzen Status assigned => feedback
2022-10-21 15:28 schanzen Assigned To schanzen => Christian Grothoff
2022-10-23 15:38 schanzen Status feedback => resolved
2022-10-23 15:38 schanzen Resolution open => fixed
2022-10-23 15:38 schanzen Fixed in Version => 0.18.0
2022-10-23 15:38 schanzen Note Added: 0019277
2022-10-31 03:22 schanzen Status resolved => closed