zeroPadder initializes StringBuilder with negative size

[icetan]

When requesting tiles with a high storage index GWC returns a 400 error with "null" as description. I found that the method zeroPadder in FilePathGenerator? when called with a number parameter with a length greater then the order parameter throws an exception. This is due to that a StringBuilder? is initialized with a negative size. I have created a patch that adds a check for how many zeros to pad and if it is zero or less then it just returns number as a string.

[icetan]

patch for zeroPadder

[arneke]

I understand the patch, but how did you trigger this, i.e. can you provide configuration file and request that results in the null?

[arneke]

Closing due to lack of feedback. Anyone should feel free to reopen if they can explain how the problem arises.

[arneke]

Got it. So it happens when you define your own grid and it's not square, which means the 2^{x assumption fails.}

I looked into passing the actual extent of the grid all the way into the path generator, but it's just too much of a hassle for directories that will noe be particularly readable for humans anyway.