To be clear, that extension also uses a "request_id" variable, accepting this patch would effectively break that extension, which I believe quite a lof of people also use.
On Wed, Apr 27, 2016 at 5:40 PM, Albert Casademont < [email protected]> wrote: > Hi all, > > We've been using this extension for quite a long time now, it's based on > UUIDs and it works flawlessly, no need to reinvent the wheel here! > > https://github.com/newobj/nginx-x-rid-header/ > > Best, > > Albert > > On Wed, Apr 27, 2016 at 5:30 PM, Maxim Dounin <[email protected]> wrote: > >> Hello! >> >> On Wed, Apr 27, 2016 at 03:23:51PM +1000, SplitIce wrote: >> >> > For ours we used a random number to start and the Process ID & Process >> > start time to try and increase uniqueness between reloads (ours is a >> 128bit >> > ID). Then applying an increment, with future requests having a higher >> id. >> > >> > Perhaps that would be better than just 128bit of random data? >> >> Using process id & start time is believed to be particularly bad >> approach in virtualized environments where multiple identical >> virtual machines and/or containers can start at the same time. >> >> -- >> Maxim Dounin >> http://nginx.org/ >> >> _______________________________________________ >> nginx-devel mailing list >> [email protected] >> http://mailman.nginx.org/mailman/listinfo/nginx-devel >> > >
_______________________________________________ nginx-devel mailing list [email protected] http://mailman.nginx.org/mailman/listinfo/nginx-devel
