On Mon, Jul 18, 2022 at 03:15:05PM +0200, Phil Sutter wrote: > On Mon, Jul 18, 2022 at 01:32:30PM +0200, Pablo Neira Ayuso wrote: > > On Sat, Jul 16, 2022 at 01:56:28PM +0200, Phil Sutter wrote: > > > Hi, > > > > > > On Sat, Jul 16, 2022 at 10:05:49AM +0200, Pablo Neira Ayuso wrote: > > > > If user specifies a too long object name, bail out. > > > > > > Shouldn't this be done in eval phase or so? As-is, this patch introduces > > > a standard syntax-specific limitation people may circumvent using JSON, > > > no? > > > > I can do it from eval phase. I will have to add more eval functions > > though, because eval is not always called from for every command. > > All I'm saying is we shouldn't divert in between the two parsers. Why is > limiting the max name length required, BTW? Right, this should be handled for the json parser too, sending v2.
